summaryrefslogtreecommitdiffstats
path: root/sound/usb
diff options
context:
space:
mode:
authorMariusz Kozlowski <m.kozlowski@tuxland.pl>2007-01-08 11:25:30 +0100
committerJaroslav Kysela <perex@suse.cz>2007-01-09 09:06:19 +0100
commit7fbe3ca571e4b0795b729658e3d76824be54cb18 (patch)
tree41cf61f75f283c770c996a5de3d51d49fe555550 /sound/usb
parent8c670714223d58086695d007b76b3e3ead54e4f5 (diff)
downloadop-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.c2
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);
OpenPOWER on IntegriCloud