ALSA: 6fire: Fix probe of multiple cards
authorTakashi Iwai <tiwai@suse.de>
Mon, 28 Oct 2013 10:24:23 +0000 (11:24 +0100)
committerBen Hutchings <ben@decadent.org.uk>
Fri, 3 Jan 2014 04:33:15 +0000 (04:33 +0000)
commit 9b389a8a022110b4bc055a19b888283544d9eba6 upstream.

The probe code of snd-usb-6fire driver overrides the devices[] pointer
wrongly without checking whether it's already occupied or not.  This
would screw up the device disconnection later.

Spotted by coverity CID 141423.

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
sound/usb/6fire/chip.c

index c7dca7b0b9fea74aa63e7162b26a57da52cc6ed5..46a2816747257b1d175d457c6dcce4a120efbe17 100644 (file)
@@ -102,7 +102,7 @@ static int __devinit usb6fire_chip_probe(struct usb_interface *intf,
                        usb_set_intfdata(intf, chips[i]);
                        mutex_unlock(&register_mutex);
                        return 0;
-               } else if (regidx < 0)
+               } else if (!devices[i] && regidx < 0)
                        regidx = i;
        }
        if (regidx < 0) {