brcmfmac: fix bug causing errorneous free on exception.
authorHante Meuleman <meuleman@broadcom.com>
Thu, 13 Sep 2012 19:12:04 +0000 (21:12 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Mon, 24 Sep 2012 18:59:09 +0000 (14:59 -0400)
This patch fixes bug where driver frees resources twice when
an exception occurs.

Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com>
Signed-off-by: Hante Meuleman <meuleman@broadcom.com>
Signed-off-by: Arend van Spriel <arend@broadcom.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/usb.c

index c6d5aeb27a0211c444b8768edc7a6885f2602c2a..03f59cd0572aeca0e0477728e6821f0040266cd0 100644 (file)
@@ -1340,10 +1340,8 @@ static int brcmf_usb_probe_cb(struct brcmf_usbdev_info *devinfo,
        struct device *dev = devinfo->dev;
 
        bus_pub = brcmf_usb_attach(devinfo, BRCMF_USB_NRXQ, BRCMF_USB_NTXQ);
-       if (!bus_pub) {
-               ret = -ENODEV;
-               goto fail;
-       }
+       if (!bus_pub)
+               return -ENODEV;
 
        bus = kzalloc(sizeof(struct brcmf_bus), GFP_ATOMIC);
        if (!bus) {