From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Date: Thu, 19 Sep 2019 21:10:01 +0000 (+0200)
Subject: 5.3-stable patches
X-Git-Tag: v4.4.194~6
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5a257b403c2023d6cbb2182b345dd3426682e58c;p=thirdparty%2Fkernel%2Fstable-queue.git

5.3-stable patches

added patches:
	media-technisat-usb2-break-out-of-loop-at-end-of-buffer.patch
---

diff --git a/queue-5.3/media-technisat-usb2-break-out-of-loop-at-end-of-buffer.patch b/queue-5.3/media-technisat-usb2-break-out-of-loop-at-end-of-buffer.patch
new file mode 100644
index 00000000000..4d4570265bf
--- /dev/null
+++ b/queue-5.3/media-technisat-usb2-break-out-of-loop-at-end-of-buffer.patch
@@ -0,0 +1,72 @@
+From 0c4df39e504bf925ab666132ac3c98d6cbbe380b Mon Sep 17 00:00:00 2001
+From: Sean Young <sean@mess.org>
+Date: Wed, 3 Jul 2019 10:52:39 -0400
+Subject: media: technisat-usb2: break out of loop at end of buffer
+
+From: Sean Young <sean@mess.org>
+
+commit 0c4df39e504bf925ab666132ac3c98d6cbbe380b upstream.
+
+Ensure we do not access the buffer beyond the end if no 0xff byte
+is encountered.
+
+Reported-by: syzbot+eaaaf38a95427be88f4b@syzkaller.appspotmail.com
+Signed-off-by: Sean Young <sean@mess.org>
+Reviewed-by: Kees Cook <keescook@chromium.org>
+Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
+Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
+
+---
+ drivers/media/usb/dvb-usb/technisat-usb2.c |   22 ++++++++++------------
+ 1 file changed, 10 insertions(+), 12 deletions(-)
+
+--- a/drivers/media/usb/dvb-usb/technisat-usb2.c
++++ b/drivers/media/usb/dvb-usb/technisat-usb2.c
+@@ -608,10 +608,9 @@ static int technisat_usb2_frontend_attac
+ static int technisat_usb2_get_ir(struct dvb_usb_device *d)
+ {
+ 	struct technisat_usb2_state *state = d->priv;
+-	u8 *buf = state->buf;
+-	u8 *b;
+-	int ret;
+ 	struct ir_raw_event ev;
++	u8 *buf = state->buf;
++	int i, ret;
+ 
+ 	buf[0] = GET_IR_DATA_VENDOR_REQUEST;
+ 	buf[1] = 0x08;
+@@ -647,26 +646,25 @@ unlock:
+ 		return 0; /* no key pressed */
+ 
+ 	/* decoding */
+-	b = buf+1;
+ 
+ #if 0
+ 	deb_rc("RC: %d ", ret);
+-	debug_dump(b, ret, deb_rc);
++	debug_dump(buf + 1, ret, deb_rc);
+ #endif
+ 
+ 	ev.pulse = 0;
+-	while (1) {
+-		ev.pulse = !ev.pulse;
+-		ev.duration = (*b * FIRMWARE_CLOCK_DIVISOR * FIRMWARE_CLOCK_TICK) / 1000;
+-		ir_raw_event_store(d->rc_dev, &ev);
+-
+-		b++;
+-		if (*b == 0xff) {
++	for (i = 1; i < ARRAY_SIZE(state->buf); i++) {
++		if (buf[i] == 0xff) {
+ 			ev.pulse = 0;
+ 			ev.duration = 888888*2;
+ 			ir_raw_event_store(d->rc_dev, &ev);
+ 			break;
+ 		}
++
++		ev.pulse = !ev.pulse;
++		ev.duration = (buf[i] * FIRMWARE_CLOCK_DIVISOR *
++			       FIRMWARE_CLOCK_TICK) / 1000;
++		ir_raw_event_store(d->rc_dev, &ev);
+ 	}
+ 
+ 	ir_raw_event_handle(d->rc_dev);
diff --git a/queue-5.3/series b/queue-5.3/series
index 883c315ab8c..1de99daf15a 100644
--- a/queue-5.3/series
+++ b/queue-5.3/series
@@ -18,3 +18,4 @@ revert-arm64-remove-unnecessary-isbs-from-set_-pte-pmd-pud.patch
 ovl-fix-regression-caused-by-overlapping-layers-detection.patch
 phy-qcom-qmp-correct-ready-status-again.patch
 floppy-fix-usercopy-direction.patch
+media-technisat-usb2-break-out-of-loop-at-end-of-buffer.patch