Skip to contentSkip to conversation

dvb-usb: gp8psk, fix potential null derefernce

PATCH In review [email protected] 1 participant 1 message Apr 27, 2010 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

PATCH 3/11 dvb-usb: gp8psk, fix potential null derefernce

From: Jiri Slaby <[email protected]>

Stanse found that in gp8psk_load_bcm4500fw there is missing a check for return value of kmalloc. Add one and bail out appropriatelly.

Cc Alan Nisota <[email protected]>
Cc Patrick Boettcher <[email protected]>
Cc Mauro Carvalho Chehab <[email protected]>

---

 drivers/media/dvb/dvb-usb/gp8psk.c |    4 ++++
 1 file changed, 4 insertions(+)
drivers/media/dvb/dvb-usb/gp8psk.c renamed from drivers/media/dvb/dvb-usb/gp8psk.c~dvb-usb-gp8psk-fix-potential-null-derefernce +4 -0
diff -puN drivers/media/dvb/dvb-usb/gp8psk.c~dvb-usb-gp8psk-fix-potential-null-derefernce drivers/media/dvb/dvb-usb/gp8psk.c
@@ -105,6 +105,10 @@ static int gp8psk_load_bcm4500fw(struct 
 
 	ptr = fw->data;
 	buf = kmalloc(64, GFP_KERNEL | GFP_DMA);
+	if (!buf) {
+		ret = -ENOMEM;
+		goto out_rel_fw;
+	}
 
 	while (ptr[0] != 0xff) {
 		u16 buflen = ptr[0] + 4;

_