ieee802154/adf7242: fix memory leak of firmware
authorSudip Mukherjee <sudipm.mukherjee@gmail.com>
Thu, 7 Apr 2016 11:16:04 +0000 (16:46 +0530)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 8 Apr 2016 17:33:30 +0000 (19:33 +0200)
If the firmware upload or the firmware verification fails then we
printed the error message and exited but we missed releasing the
firmware.

Signed-off-by: Sudip Mukherjee <sudip.mukherjee@codethink.co.uk>
Acked-by: Michael Hennerich <michael.hennerich@analog.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
drivers/net/ieee802154/adf7242.c

index 89154c0797888694a082141e07d8f9c8c973c37e..b82e39d2439495641fb9edc8559efbb2fcf7b2a8 100644 (file)
@@ -1030,6 +1030,7 @@ static int adf7242_hw_init(struct adf7242_local *lp)
        if (ret) {
                dev_err(&lp->spi->dev,
                        "upload firmware failed with %d\n", ret);
+               release_firmware(fw);
                return ret;
        }
 
@@ -1037,6 +1038,7 @@ static int adf7242_hw_init(struct adf7242_local *lp)
        if (ret) {
                dev_err(&lp->spi->dev,
                        "verify firmware failed with %d\n", ret);
+               release_firmware(fw);
                return ret;
        }