Преглед изворни кода

brcm80211: fmac: small memory leak on error

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>
Dan Carpenter пре 13 година
родитељ
комит
ad9547c041
1 измењених фајлова са 3 додато и 1 уклоњено
  1. 3 1
      drivers/net/wireless/brcm80211/brcmfmac/bcmsdh_sdmmc.c

+ 3 - 1
drivers/net/wireless/brcm80211/brcmfmac/bcmsdh_sdmmc.c

@@ -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;