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 | |
| parent | [ALSA] _snd_cmipci_uswitch_put doesn't set zero flags (diff) | |
| download | linux-7fbe3ca571e4b0795b729658e3d76824be54cb18.tar.xz linux-7fbe3ca571e4b0795b729658e3d76824be54cb18.zip | |
[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')
| -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 e74eb1bc8d87..7b3bf3545a3b 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); |