diff options
author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2007-01-08 11:25:30 +0100 |
---|---|---|
committer | Jaroslav Kysela <perex@suse.cz> | 2007-01-09 09:06:19 +0100 |
commit | 7fbe3ca571e4b0795b729658e3d76824be54cb18 (patch) | |
tree | 41cf61f75f283c770c996a5de3d51d49fe555550 /sound/usb | |
parent | 8c670714223d58086695d007b76b3e3ead54e4f5 (diff) | |
download | op-kernel-dev-7fbe3ca571e4b0795b729658e3d76824be54cb18.zip op-kernel-dev-7fbe3ca571e4b0795b729658e3d76824be54cb18.tar.gz |
[ALSA] usb: usbmixer error path fix
Without the patch below namelist[0] will not be freed in case
of kmalloc error.
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>
Diffstat (limited to 'sound/usb')
-rw-r--r-- | sound/usb/usbmixer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/usb/usbmixer.c b/sound/usb/usbmixer.c index e74eb1b..7b3bf35 100644 --- a/sound/usb/usbmixer.c +++ b/sound/usb/usbmixer.c @@ -1526,7 +1526,7 @@ static int parse_audio_selector_unit(struct mixer_build *state, int unitid, unsi namelist[i] = kmalloc(MAX_ITEM_NAME_LEN, GFP_KERNEL); if (! namelist[i]) { snd_printk(KERN_ERR "cannot malloc\n"); - while (--i > 0) + while (i--) kfree(namelist[i]); kfree(namelist); kfree(cval); |