[media] lmedm04: fix data usage past the end of the buffer
authorMalcolm Priestley <tvboxspy@gmail.com>
Mon, 13 Aug 2012 17:42:17 +0000 (14:42 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Mon, 13 Aug 2012 22:53:29 +0000 (19:53 -0300)
On Mon, 2012-08-13 at 19:58 +0300, Dan Carpenter wrote:
> Hello Mauro Carvalho Chehab,
>
> The patch db6651a9ebb3: "[media] lmedm04: fix build" from Aug 12,
> 2012, leads to the following warning:
> drivers/media/dvb/dvb-usb-v2/lmedm04.c:769 lme2510_download_firmware()
>   error: usb_control_msg() 'data' too small (128 vs 265)
>
>    737          data = kzalloc(128, GFP_KERNEL);
>                                ^^^
> data is 128 bytes.

Control isn't used, so remove it.

Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Malcolm Priestley <tvboxspy@gmail.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/dvb-usb-v2/lmedm04.c

index c6bc1b8b58b8c84b6b8f610dc07c208571384680..c41d9d9ec7b54df963ede5ad9deb6d9534e50620 100644 (file)
@@ -766,10 +766,6 @@ static int lme2510_download_firmware(struct dvb_usb_device *d,
                }
        }
 
-       usb_control_msg(d->udev, usb_rcvctrlpipe(d->udev, 0),
-                       0x06, 0x80, 0x0200, 0x00, data, 0x0109, 1000);
-
-
        data[0] = 0x8a;
        len_in = 1;
        msleep(2000);