brcm80211: fmac: small memory leak on error
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 30 Nov 2011 08:49:38 +0000 (11:49 +0300)
committerJohn W. Linville <linville@tuxdriver.com>
Wed, 30 Nov 2011 20:08:59 +0000 (15:08 -0500)
We should free "bus_if" here, it's a small leak but it makes the static
checkers happy.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/brcm80211/brcmfmac/bcmsdh_sdmmc.c

index 77f84f8..b416e27 100644 (file)
@@ -477,8 +477,10 @@ static int brcmf_ops_sdio_probe(struct sdio_func *func,
                if (!bus_if)
                        return -ENOMEM;
                sdiodev = kzalloc(sizeof(struct brcmf_sdio_dev), GFP_KERNEL);
-               if (!sdiodev)
+               if (!sdiodev) {
+                       kfree(bus_if);
                        return -ENOMEM;
+               }
                sdiodev->dev = &func->card->dev;
                sdiodev->func[0] = func->card->sdio_func[0];
                sdiodev->func[1] = func;