diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2011-05-30 12:49:01 +0200 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2011-06-02 19:56:31 +0200 |
| commit | bf0be0e951cf1c4c9ce38032195cd8095a16d828 (patch) | |
| tree | 0f32e4bc1dd584e0f5e4cae2ceb7acdc0e74a61e /sound/usb/6fire | |
| parent | ALSA: fix hda AZX_DCAPS_NO_TCSEL quirk check in driver_caps (diff) | |
| download | linux-bf0be0e951cf1c4c9ce38032195cd8095a16d828.tar.xz linux-bf0be0e951cf1c4c9ce38032195cd8095a16d828.zip | |
ALSA: 6fire: Don't leak firmware in error path
One of the error paths in
sound/usb/6fire/firmware.c::usb6fire_fw_ezusb_upload() neglects to free
the memory allocated for the firmware before returning, thus leaking the
memory.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/usb/6fire')
| -rw-r--r-- | sound/usb/6fire/firmware.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/sound/usb/6fire/firmware.c b/sound/usb/6fire/firmware.c index d47beffedb0f..a91719d5918b 100644 --- a/sound/usb/6fire/firmware.c +++ b/sound/usb/6fire/firmware.c @@ -227,6 +227,7 @@ static int usb6fire_fw_ezusb_upload( ret = usb6fire_fw_ihex_init(fw, rec); if (ret < 0) { kfree(rec); + release_firmware(fw); snd_printk(KERN_ERR PREFIX "error validating ezusb " "firmware %s.\n", fwname); return ret; |