]> git.karo-electronics.de Git - karo-tx-linux.git/commit
[media] media, Micronas dvb-t: Fix mem leaks, don't needlessly zero mem, fix spelling
authorJesper Juhl <jj@chaosbits.net>
Wed, 13 Jul 2011 20:58:13 +0000 (17:58 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 13 Jul 2011 23:32:06 +0000 (20:32 -0300)
commita87ee6f2fc48a791f1f4470deb052e7e6302f2f4
treea531cc549cc4ff7eb6c018c710c87ee08c10b06f
parent4c7cf3efbe8504aa28e299db1b597f49c2b8e044
[media] media, Micronas dvb-t: Fix mem leaks, don't needlessly zero mem, fix spelling

In drivers/media/dvb/frontends/drxd_hard.c::load_firmware() I see 3
small issues:

 1) When the 'fw' variable goes out of scope we'll leak the memory
 allocated to it by request_firmware() by neglecting to call
 release_firmware().

 2) After a successful request_firmware() we allocate fw->size bytes
 of memory using kzalloc() only to immediately overwrite all that
 memory with memcpy(), so asking for zeroed memory seems like wasted
 effort - just use kmalloc().

 3) In one of the error messages "no memory" lacks a space and is
 written as "nomemory".

This patch fixes all 3 issues.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/frontends/drxd_hard.c