Re: [PATCH] sound/oss/msnd_pinnacle.c: add vfrees

From: Takashi Iwai
Date: Tue Apr 10 2012 - 02:42:56 EST


At Mon, 9 Apr 2012 10:16:32 +0200,
Julia Lawall wrote:
>
> From: Julia Lawall <Julia.Lawall@xxxxxxx>
>
> At the point of this error-handling code, HAVE_DSPCODEH may be undefined,
> so free INITCODE and PERMCODE as done elsewhere. A jump and label are
> introduced to avoid code duplication.
>
> Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx>

Thanks, applied now.


Takashi

>
> ---
> sound/oss/msnd_pinnacle.c | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/sound/oss/msnd_pinnacle.c b/sound/oss/msnd_pinnacle.c
> index 2c79d60..536c4c0 100644
> --- a/sound/oss/msnd_pinnacle.c
> +++ b/sound/oss/msnd_pinnacle.c
> @@ -1294,6 +1294,8 @@ static int __init calibrate_adc(WORD srate)
>
> static int upload_dsp_code(void)
> {
> + int ret = 0;
> +
> msnd_outb(HPBLKSEL_0, dev.io + HP_BLKS);
> #ifndef HAVE_DSPCODEH
> INITCODESIZE = mod_firmware_load(INITCODEFILE, &INITCODE);
> @@ -1312,7 +1314,8 @@ static int upload_dsp_code(void)
> memcpy_toio(dev.base, PERMCODE, PERMCODESIZE);
> if (msnd_upload_host(&dev, INITCODE, INITCODESIZE) < 0) {
> printk(KERN_WARNING LOGNAME ": Error uploading to DSP\n");
> - return -ENODEV;
> + ret = -ENODEV;
> + goto out;
> }
> #ifdef HAVE_DSPCODEH
> printk(KERN_INFO LOGNAME ": DSP firmware uploaded (resident)\n");
> @@ -1320,12 +1323,13 @@ static int upload_dsp_code(void)
> printk(KERN_INFO LOGNAME ": DSP firmware uploaded\n");
> #endif
>
> +out:
> #ifndef HAVE_DSPCODEH
> vfree(INITCODE);
> vfree(PERMCODE);
> #endif
>
> - return 0;
> + return ret;
> }
>
> #ifdef MSND_CLASSIC
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/