From: Dan Carpenter Date: Fri, 5 Sep 2014 12:09:28 +0000 (-0300) Subject: media: ttusb-dec: buffer overflow in ioctl X-Git-Tag: upstream/snapshot3+hdmi~17^2~50 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c8e0fd4818f29aaafafb01f0bacf376b86e82830;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git media: ttusb-dec: buffer overflow in ioctl commit f2e323ec96077642d397bb1c355def536d489d16 upstream. We need to add a limit check here so we don't overflow the buffer. Signed-off-by: Dan Carpenter Signed-off-by: Mauro Carvalho Chehab Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/media/usb/ttusb-dec/ttusbdecfe.c b/drivers/media/usb/ttusb-dec/ttusbdecfe.c index 5c45c9d..9c29552 100644 --- a/drivers/media/usb/ttusb-dec/ttusbdecfe.c +++ b/drivers/media/usb/ttusb-dec/ttusbdecfe.c @@ -156,6 +156,9 @@ static int ttusbdecfe_dvbs_diseqc_send_master_cmd(struct dvb_frontend* fe, struc 0x00, 0x00, 0x00, 0x00, 0x00, 0x00 }; + if (cmd->msg_len > sizeof(b) - 4) + return -EINVAL; + memcpy(&b[4], cmd->msg, cmd->msg_len); state->config->send_command(fe, 0x72,