diff options
author | Jesper Juhl <jj@chaosbits.net> | 2011-02-06 15:49:02 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2011-03-21 19:32:22 -0400 |
commit | ba0fd56a4fbb92999554a2e417924796c2ad2f8f (patch) | |
tree | 9939ba6b4d1d450c650fb159ed2c7031924ebafa /drivers/media/dvb/ttusb-budget | |
parent | 19a0a296f2df31c24f87fd3e7ee7f2640e49e5d4 (diff) |
[media] TTUSB DVB: ttusb_boot_dsp() needs to release_firmware() or it leaks memory
In drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c:ttusb_boot_dsp() we
need to call release_firmware(fw) before returning or we'll leak - no
matter if we succeed or fail.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb/ttusb-budget')
-rw-r--r-- | drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c index 40625b26ac1..cbe2f0de144 100644 --- a/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c +++ b/drivers/media/dvb/ttusb-budget/dvb-ttusb-budget.c | |||
@@ -334,6 +334,7 @@ static int ttusb_boot_dsp(struct ttusb *ttusb) | |||
334 | err = ttusb_cmd(ttusb, b, 4, 0); | 334 | err = ttusb_cmd(ttusb, b, 4, 0); |
335 | 335 | ||
336 | done: | 336 | done: |
337 | release_firmware(fw); | ||
337 | if (err) { | 338 | if (err) { |
338 | dprintk("%s: usb_bulk_msg() failed, return value %i!\n", | 339 | dprintk("%s: usb_bulk_msg() failed, return value %i!\n", |
339 | __func__, err); | 340 | __func__, err); |