rt2x00: Don't leak mem in error path of rt2x00lib_request_firmware()
authorJesper Juhl <jj@chaosbits.net>
Mon, 10 Jan 2011 23:47:44 +0000 (00:47 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 13 Jan 2011 20:46:45 +0000 (15:46 -0500)
We need to release_firmware() in order not to leak memory.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Ivo van Doorn <IvDoorn@gmail.com>
Acked-by: Pekka Enberg <penberg@kernel.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rt2x00/rt2x00firmware.c

index f0e1eb72befc0abff02c5f7e2f33d8d610c70cdd..be0ff78c1b1666178864c62e8f367a6fae939a9a 100644 (file)
@@ -58,6 +58,7 @@ static int rt2x00lib_request_firmware(struct rt2x00_dev *rt2x00dev)
 
        if (!fw || !fw->size || !fw->data) {
                ERROR(rt2x00dev, "Failed to read Firmware.\n");
+               release_firmware(fw);
                return -ENOENT;
        }