Hello,

        Without the patch below namelist[0] will not be freed in case
of kmalloc error.

Signed-off-by: Mariusz Kozlowski <[EMAIL PROTECTED]>

 sound/usb/usbmixer.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff -upr linux-2.6.20-rc2-mm1-a/sound/usb/usbmixer.c 
linux-2.6.20-rc2-mm1-b/sound/usb/usbmixer.c
--- linux-2.6.20-rc2-mm1-a/sound/usb/usbmixer.c 2006-12-24 05:00:32.000000000 
+0100
+++ linux-2.6.20-rc2-mm1-b/sound/usb/usbmixer.c 2007-01-01 23:55:31.000000000 
+0100
@@ -1526,7 +1526,7 @@ static int parse_audio_selector_unit(str
                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);


-- 
Regards,

        Mariusz Kozlowski

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to