|
Vasant Karasulli |
af7b7e |
From 0ed554fd769a19ea8464bb83e9ac201002ef74ad Mon Sep 17 00:00:00 2001
|
|
Vasant Karasulli |
af7b7e |
From: Baisong Zhong <zhongbaisong@huawei.com>
|
|
Vasant Karasulli |
af7b7e |
Date: Sun, 20 Nov 2022 06:59:18 +0000
|
|
Vasant Karasulli |
af7b7e |
Subject: [PATCH] media: dvb-usb: az6027: fix null-ptr-deref in
|
|
Vasant Karasulli |
af7b7e |
az6027_i2c_xfer()
|
|
Vasant Karasulli |
af7b7e |
Git-commit: 0ed554fd769a19ea8464bb83e9ac201002ef74ad
|
|
Vasant Karasulli |
af7b7e |
Patch-mainline: v6.2-rc1
|
|
Vasant Karasulli |
af7b7e |
References: bsc#1209291 CVE-2023-28328
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
Wei Chen reports a kernel bug as blew:
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
general protection fault, probably for non-canonical address
|
|
Vasant Karasulli |
af7b7e |
Kasan: null-ptr-deref in range [0x0000000000000010-0x0000000000000017]
|
|
Vasant Karasulli |
af7b7e |
...
|
|
Vasant Karasulli |
af7b7e |
Call Trace:
|
|
Vasant Karasulli |
af7b7e |
<TASK>
|
|
Vasant Karasulli |
af7b7e |
__i2c_transfer+0x77e/0x1930 drivers/i2c/i2c-core-base.c:2109
|
|
Vasant Karasulli |
af7b7e |
i2c_transfer+0x1d5/0x3d0 drivers/i2c/i2c-core-base.c:2170
|
|
Vasant Karasulli |
af7b7e |
i2cdev_ioctl_rdwr+0x393/0x660 drivers/i2c/i2c-dev.c:297
|
|
Vasant Karasulli |
af7b7e |
i2cdev_ioctl+0x75d/0x9f0 drivers/i2c/i2c-dev.c:458
|
|
Vasant Karasulli |
af7b7e |
vfs_ioctl fs/ioctl.c:51 [inline]
|
|
Vasant Karasulli |
af7b7e |
__do_sys_ioctl fs/ioctl.c:870 [inline]
|
|
Vasant Karasulli |
af7b7e |
__se_sys_ioctl+0xfb/0x170 fs/ioctl.c:856
|
|
Vasant Karasulli |
af7b7e |
do_syscall_x64 arch/x86/entry/common.c:50 [inline]
|
|
Vasant Karasulli |
af7b7e |
do_syscall_64+0x3d/0x90 arch/x86/entry/common.c:80
|
|
Vasant Karasulli |
af7b7e |
entry_SYSCALL_64_after_hwframe+0x63/0xcd
|
|
Vasant Karasulli |
af7b7e |
Rip: 0033:0x7fd834a8bded
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
In az6027_i2c_xfer(), if msg[i].addr is 0x99,
|
|
Vasant Karasulli |
af7b7e |
a null-ptr-deref will caused when accessing msg[i].buf.
|
|
Vasant Karasulli |
af7b7e |
For msg[i].len is 0 and msg[i].buf is null.
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
Fix this by checking msg[i].len in az6027_i2c_xfer().
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
Link: https://lore.kernel.org/lkml/CAO4mrfcPHB5aQJO=mpqV+p8mPLNg-Fok0gw8gZ=zemAfMGTzMg@mail.gmail.com/
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
Link: https://lore.kernel.org/linux-media/20221120065918.2160782-1-zhongbaisong@huawei.com
|
|
Vasant Karasulli |
af7b7e |
Fixes: 76f9a820c867 ("V4L/DVB: AZ6027: Initial import of the driver")
|
|
Vasant Karasulli |
af7b7e |
Reported-by: Wei Chen <harperchen1110@gmail.com>
|
|
Vasant Karasulli |
af7b7e |
Signed-off-by: Baisong Zhong <zhongbaisong@huawei.com>
|
|
Vasant Karasulli |
af7b7e |
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
|
|
Vasant Karasulli |
af7b7e |
Acked-by: Vasant Karasulli <vkarasulli@suse.de>
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
---
|
|
Vasant Karasulli |
af7b7e |
drivers/media/usb/dvb-usb/az6027.c | 4 ++++
|
|
Vasant Karasulli |
af7b7e |
1 file changed, 4 insertions(+)
|
|
Vasant Karasulli |
af7b7e |
|
|
Vasant Karasulli |
af7b7e |
diff --git a/drivers/media/usb/dvb-usb/az6027.c b/drivers/media/usb/dvb-usb/az6027.c
|
|
Vasant Karasulli |
af7b7e |
index cf15988dfb51..7d78ee09be5e 100644
|
|
Vasant Karasulli |
af7b7e |
--- a/drivers/media/usb/dvb-usb/az6027.c
|
|
Vasant Karasulli |
af7b7e |
+++ b/drivers/media/usb/dvb-usb/az6027.c
|
|
Vasant Karasulli |
af7b7e |
@@ -975,6 +975,10 @@ static int az6027_i2c_xfer(struct i2c_adapter *adap, struct i2c_msg msg[], int n
|
|
Vasant Karasulli |
af7b7e |
if (msg[i].addr == 0x99) {
|
|
Vasant Karasulli |
af7b7e |
req = 0xBE;
|
|
Vasant Karasulli |
af7b7e |
index = 0;
|
|
Vasant Karasulli |
af7b7e |
+ if (msg[i].len < 1) {
|
|
Vasant Karasulli |
af7b7e |
+ i = -EOPNOTSUPP;
|
|
Vasant Karasulli |
af7b7e |
+ break;
|
|
Vasant Karasulli |
af7b7e |
+ }
|
|
Vasant Karasulli |
af7b7e |
value = msg[i].buf[0] & 0x00ff;
|
|
Vasant Karasulli |
af7b7e |
length = 1;
|
|
Vasant Karasulli |
af7b7e |
az6027_usb_out_op(d, req, value, index, data, length);
|
|
Vasant Karasulli |
af7b7e |
--
|
|
Vasant Karasulli |
af7b7e |
2.34.1
|
|
Vasant Karasulli |
af7b7e |
|