Madge Ambassador ATM Adapter driver: Always release_firmware() in ucode_init() and...
authorJesper Juhl <jj@chaosbits.net>
Sun, 9 Jan 2011 11:32:38 +0000 (11:32 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 9 Jan 2011 23:45:48 +0000 (15:45 -0800)
commit6a75da4a1a1d220fc3e32b816e83c258617505c5
treec7cc5931f5f5637e19892635cdf30ad2bcd8d622
parent26877c794407855f65c056d7914b2e72ae9b93b8
Madge Ambassador ATM Adapter driver: Always release_firmware() in ucode_init() and don't leak memory.

Failure to call release_firmware() will result in memory leak in
drivers/atm/ambassador.c::ucode_init().
This patch makes sure we always call release_firmware() when needed,
thus removing the leak(s).

Yes, I know checkpatch complains about this patch, but it was either that
or completely mess up the existing style, so I opted to use the existing
style and live with the checkpatch related flak.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/atm/ambassador.c