* [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
@ 2021-12-18 11:07 syzbot
2021-12-18 21:14 ` Pavel Skripkin
0 siblings, 1 reply; 7+ messages in thread
From: syzbot @ 2021-12-18 11:07 UTC (permalink / raw)
To: andrew, davem, glider, kuba, linux-kernel, linux-usb, linux,
netdev, paskripkin, syzkaller-bugs
Hello,
syzbot found the following issue on:
HEAD commit: b0a8b5053e8b kmsan: core: add dependency on DEBUG_KERNEL
git tree: https://github.com/google/kmsan.git master
console output: https://syzkaller.appspot.com/x/log.txt?x=13a4d133b00000
kernel config: https://syzkaller.appspot.com/x/.config?x=46a956fc7a887c60
dashboard link: https://syzkaller.appspot.com/bug?extid=f44badb06036334e867a
compiler: clang version 14.0.0 (/usr/local/google/src/llvm-git-monorepo 2b554920f11c8b763cd9ed9003f4e19b919b8e1f), GNU ld (GNU Binutils for Debian) 2.35.2
syz repro: https://syzkaller.appspot.com/x/repro.syz?x=149fddcbb00000
C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17baef25b00000
IMPORTANT: if you fix the issue, please add the following tag to the commit:
Reported-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
=====================================================
BUG: KMSAN: uninit-value in asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline]
BUG: KMSAN: uninit-value in asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline] drivers/net/usb/asix_common.c:497
BUG: KMSAN: uninit-value in asix_mdio_read+0x3c1/0xb00 drivers/net/usb/asix_common.c:497 drivers/net/usb/asix_common.c:497
asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline]
asix_check_host_enable drivers/net/usb/asix_common.c:82 [inline] drivers/net/usb/asix_common.c:497
asix_mdio_read+0x3c1/0xb00 drivers/net/usb/asix_common.c:497 drivers/net/usb/asix_common.c:497
asix_mdio_bus_read+0xba/0xe0 drivers/net/usb/asix_common.c:556 drivers/net/usb/asix_common.c:556
__mdiobus_read+0xbf/0x4f0 drivers/net/phy/mdio_bus.c:755 drivers/net/phy/mdio_bus.c:755
mdiobus_read+0xaa/0xf0 drivers/net/phy/mdio_bus.c:862 drivers/net/phy/mdio_bus.c:862
get_phy_c22_id drivers/net/phy/phy_device.c:813 [inline]
get_phy_c22_id drivers/net/phy/phy_device.c:813 [inline] drivers/net/phy/phy_device.c:890
get_phy_device+0x218/0x8b0 drivers/net/phy/phy_device.c:890 drivers/net/phy/phy_device.c:890
mdiobus_scan+0x1c7/0x940
__mdiobus_register+0xe16/0x1200 drivers/net/phy/mdio_bus.c:583 drivers/net/phy/mdio_bus.c:583
__devm_mdiobus_register+0x18f/0x2f0 drivers/net/phy/mdio_devres.c:87 drivers/net/phy/mdio_devres.c:87
ax88772_init_mdio drivers/net/usb/asix_devices.c:676 [inline]
ax88772_init_mdio drivers/net/usb/asix_devices.c:676 [inline] drivers/net/usb/asix_devices.c:786
ax88772_bind+0x10b1/0x1770 drivers/net/usb/asix_devices.c:786 drivers/net/usb/asix_devices.c:786
usbnet_probe+0x1284/0x4140 drivers/net/usb/usbnet.c:1747 drivers/net/usb/usbnet.c:1747
usb_probe_interface+0xf19/0x1600 drivers/usb/core/driver.c:396 drivers/usb/core/driver.c:396
really_probe+0x67d/0x1510 drivers/base/dd.c:596 drivers/base/dd.c:596
__driver_probe_device+0x3e9/0x530 drivers/base/dd.c:751 drivers/base/dd.c:751
driver_probe_device drivers/base/dd.c:781 [inline]
driver_probe_device drivers/base/dd.c:781 [inline] drivers/base/dd.c:898
__device_attach_driver+0x79f/0x1120 drivers/base/dd.c:898 drivers/base/dd.c:898
bus_for_each_drv+0x2d6/0x3f0 drivers/base/bus.c:427 drivers/base/bus.c:427
__device_attach+0x593/0x8e0 drivers/base/dd.c:969 drivers/base/dd.c:969
device_initial_probe+0x4a/0x60 drivers/base/dd.c:1016 drivers/base/dd.c:1016
bus_probe_device+0x17b/0x3e0 drivers/base/bus.c:487 drivers/base/bus.c:487
device_add+0x1d3e/0x2400 drivers/base/core.c:3394 drivers/base/core.c:3394
usb_set_configuration+0x37e9/0x3ed0 drivers/usb/core/message.c:2170 drivers/usb/core/message.c:2170
usb_generic_driver_probe+0x13c/0x300 drivers/usb/core/generic.c:238 drivers/usb/core/generic.c:238
usb_probe_device+0x309/0x570 drivers/usb/core/driver.c:293 drivers/usb/core/driver.c:293
really_probe+0x67d/0x1510 drivers/base/dd.c:596 drivers/base/dd.c:596
__driver_probe_device+0x3e9/0x530 drivers/base/dd.c:751 drivers/base/dd.c:751
driver_probe_device drivers/base/dd.c:781 [inline]
driver_probe_device drivers/base/dd.c:781 [inline] drivers/base/dd.c:898
__device_attach_driver+0x79f/0x1120 drivers/base/dd.c:898 drivers/base/dd.c:898
bus_for_each_drv+0x2d6/0x3f0 drivers/base/bus.c:427 drivers/base/bus.c:427
__device_attach+0x593/0x8e0 drivers/base/dd.c:969 drivers/base/dd.c:969
device_initial_probe+0x4a/0x60 drivers/base/dd.c:1016 drivers/base/dd.c:1016
bus_probe_device+0x17b/0x3e0 drivers/base/bus.c:487 drivers/base/bus.c:487
device_add+0x1d3e/0x2400 drivers/base/core.c:3394 drivers/base/core.c:3394
usb_new_device+0x1b8e/0x2950 drivers/usb/core/hub.c:2563 drivers/usb/core/hub.c:2563
hub_port_connect drivers/usb/core/hub.c:5353 [inline]
hub_port_connect_change drivers/usb/core/hub.c:5497 [inline]
port_event drivers/usb/core/hub.c:5643 [inline]
hub_port_connect drivers/usb/core/hub.c:5353 [inline] drivers/usb/core/hub.c:5725
hub_port_connect_change drivers/usb/core/hub.c:5497 [inline] drivers/usb/core/hub.c:5725
port_event drivers/usb/core/hub.c:5643 [inline] drivers/usb/core/hub.c:5725
hub_event+0x5ad2/0x8910 drivers/usb/core/hub.c:5725 drivers/usb/core/hub.c:5725
process_one_work+0xdb9/0x1820 kernel/workqueue.c:2298 kernel/workqueue.c:2298
process_scheduled_works kernel/workqueue.c:2361 [inline]
process_scheduled_works kernel/workqueue.c:2361 [inline] kernel/workqueue.c:2447
worker_thread+0x1735/0x21f0 kernel/workqueue.c:2447 kernel/workqueue.c:2447
kthread+0x721/0x850 kernel/kthread.c:327 kernel/kthread.c:327
ret_from_fork+0x1f/0x30
Local variable smsr.i created at:
asix_mdio_read+0xbc/0xb00 drivers/net/usb/asix_common.c:497 drivers/net/usb/asix_common.c:497
asix_mdio_bus_read+0xba/0xe0 drivers/net/usb/asix_common.c:556 drivers/net/usb/asix_common.c:556
CPU: 0 PID: 3145 Comm: kworker/0:3 Not tainted 5.16.0-rc5-syzkaller #0
Hardware name: Google Google Compute Engine/Google Compute Engine, BIOS Google 01/01/2011
Workqueue: usb_hub_wq hub_event
=====================================================
---
This report is generated by a bot. It may contain errors.
See https://goo.gl/tpsmEJ for more information about syzbot.
syzbot engineers can be reached at syzkaller@googlegroups.com.
syzbot will keep track of this issue. See:
https://goo.gl/tpsmEJ#status for how to communicate with syzbot.
syzbot can test patches for this issue, for details see:
https://goo.gl/tpsmEJ#testing-patches
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-18 11:07 [syzbot] KMSAN: uninit-value in asix_mdio_read (2) syzbot
@ 2021-12-18 21:14 ` Pavel Skripkin
2021-12-18 21:42 ` Andrew Lunn
2021-12-19 1:03 ` syzbot
0 siblings, 2 replies; 7+ messages in thread
From: Pavel Skripkin @ 2021-12-18 21:14 UTC (permalink / raw)
To: syzbot, andrew, davem, glider, kuba, linux-kernel, linux-usb,
linux, netdev, syzkaller-bugs
[-- Attachment #1: Type: text/plain, Size: 1079 bytes --]
On 12/18/21 14:07, syzbot wrote:
> Hello,
>
> syzbot found the following issue on:
>
> HEAD commit: b0a8b5053e8b kmsan: core: add dependency on DEBUG_KERNEL
> git tree: https://github.com/google/kmsan.git master
> console output: https://syzkaller.appspot.com/x/log.txt?x=13a4d133b00000
> kernel config: https://syzkaller.appspot.com/x/.config?x=46a956fc7a887c60
> dashboard link: https://syzkaller.appspot.com/bug?extid=f44badb06036334e867a
> compiler: clang version 14.0.0 (/usr/local/google/src/llvm-git-monorepo 2b554920f11c8b763cd9ed9003f4e19b919b8e1f), GNU ld (GNU Binutils for Debian) 2.35.2
> syz repro: https://syzkaller.appspot.com/x/repro.syz?x=149fddcbb00000
> C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17baef25b00000
>
> IMPORTANT: if you fix the issue, please add the following tag to the commit:
> Reported-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
>
The last unhanded case is asix_read_cmd() == 0. Let's handle it...
#syz test: https://github.com/google/kmsan.git master
With regards,
Pavel Skripkin
[-- Attachment #2: ph --]
[-- Type: text/plain, Size: 436 bytes --]
diff --git a/drivers/net/usb/asix_common.c b/drivers/net/usb/asix_common.c
index 42ba4af68090..fb4f6ce9466a 100644
--- a/drivers/net/usb/asix_common.c
+++ b/drivers/net/usb/asix_common.c
@@ -77,7 +77,7 @@ static int asix_check_host_enable(struct usbnet *dev, int in_pm)
0, 0, 1, &smsr, in_pm);
if (ret == -ENODEV)
break;
- else if (ret < 0)
+ else if (ret <= 0)
continue;
else if (smsr & AX_HOST_EN)
break;
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-18 21:14 ` Pavel Skripkin
@ 2021-12-18 21:42 ` Andrew Lunn
2021-12-18 22:01 ` Pavel Skripkin
2021-12-19 1:03 ` syzbot
1 sibling, 1 reply; 7+ messages in thread
From: Andrew Lunn @ 2021-12-18 21:42 UTC (permalink / raw)
To: Pavel Skripkin
Cc: syzbot, davem, glider, kuba, linux-kernel, linux-usb, linux,
netdev, syzkaller-bugs
On Sun, Dec 19, 2021 at 12:14:30AM +0300, Pavel Skripkin wrote:
> On 12/18/21 14:07, syzbot wrote:
> > Hello,
> >
> > syzbot found the following issue on:
> >
> > HEAD commit: b0a8b5053e8b kmsan: core: add dependency on DEBUG_KERNEL
> > git tree: https://github.com/google/kmsan.git master
> > console output: https://syzkaller.appspot.com/x/log.txt?x=13a4d133b00000
> > kernel config: https://syzkaller.appspot.com/x/.config?x=46a956fc7a887c60
> > dashboard link: https://syzkaller.appspot.com/bug?extid=f44badb06036334e867a
> > compiler: clang version 14.0.0 (/usr/local/google/src/llvm-git-monorepo 2b554920f11c8b763cd9ed9003f4e19b919b8e1f), GNU ld (GNU Binutils for Debian) 2.35.2
> > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=149fddcbb00000
> > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17baef25b00000
> >
> > IMPORTANT: if you fix the issue, please add the following tag to the commit:
> > Reported-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
> >
>
> The last unhanded case is asix_read_cmd() == 0. Let's handle it...
That does not look correct, i think.
asix_read_cmd() == 0 means no error from the read itself. If there is
no error, we should look at the value of smsr and test for bit
AX_HOST_EN. Doing a continue means we just ignore the value returned
by the good read.
I think the correct fix is to look at the value of i. If we have
exceeded 30, we should return -ETIMEDOUT.
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-18 21:42 ` Andrew Lunn
@ 2021-12-18 22:01 ` Pavel Skripkin
0 siblings, 0 replies; 7+ messages in thread
From: Pavel Skripkin @ 2021-12-18 22:01 UTC (permalink / raw)
To: Andrew Lunn
Cc: syzbot, davem, glider, kuba, linux-kernel, linux-usb, linux,
netdev, syzkaller-bugs
On 12/19/21 00:42, Andrew Lunn wrote:
> On Sun, Dec 19, 2021 at 12:14:30AM +0300, Pavel Skripkin wrote:
>> On 12/18/21 14:07, syzbot wrote:
>> > Hello,
>> >
>> > syzbot found the following issue on:
>> >
>> > HEAD commit: b0a8b5053e8b kmsan: core: add dependency on DEBUG_KERNEL
>> > git tree: https://github.com/google/kmsan.git master
>> > console output: https://syzkaller.appspot.com/x/log.txt?x=13a4d133b00000
>> > kernel config: https://syzkaller.appspot.com/x/.config?x=46a956fc7a887c60
>> > dashboard link: https://syzkaller.appspot.com/bug?extid=f44badb06036334e867a
>> > compiler: clang version 14.0.0 (/usr/local/google/src/llvm-git-monorepo 2b554920f11c8b763cd9ed9003f4e19b919b8e1f), GNU ld (GNU Binutils for Debian) 2.35.2
>> > syz repro: https://syzkaller.appspot.com/x/repro.syz?x=149fddcbb00000
>> > C reproducer: https://syzkaller.appspot.com/x/repro.c?x=17baef25b00000
>> >
>> > IMPORTANT: if you fix the issue, please add the following tag to the commit:
>> > Reported-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
>> >
>>
>> The last unhanded case is asix_read_cmd() == 0. Let's handle it...
>
> That does not look correct, i think.
>
> asix_read_cmd() == 0 means no error from the read itself. If there is
> no error, we should look at the value of smsr and test for bit
As far as I understand asix_read_cmd() == 0 means 0 bytes was read,
since it just returns usb_control_msg() return value. I don't know if
usb_control_msg() can ever return 0, but looks like it's the only
unhanded case
> AX_HOST_EN. Doing a continue means we just ignore the value returned
> by the good read.
>
> I think the correct fix is to look at the value of i. If we have
> exceeded 30, we should return -ETIMEDOUT.
>
Sorry, I don't get how it can help. Report says, that on following line
else if (smsr & AX_HOST_EN)
smsr is uninitialized. How value of i can help here?
Can you, please, explain if you don't mind :)
Thanks
With regards,
Pavel Skripkin
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-18 21:14 ` Pavel Skripkin
2021-12-18 21:42 ` Andrew Lunn
@ 2021-12-19 1:03 ` syzbot
2021-12-19 17:17 ` Andrew Lunn
1 sibling, 1 reply; 7+ messages in thread
From: syzbot @ 2021-12-19 1:03 UTC (permalink / raw)
To: andrew, davem, glider, kuba, linux-kernel, linux-usb, linux,
netdev, paskripkin, syzkaller-bugs
Hello,
syzbot has tested the proposed patch and the reproducer did not trigger any issue:
Reported-and-tested-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
Tested on:
commit: b0a8b505 kmsan: core: add dependency on DEBUG_KERNEL
git tree: https://github.com/google/kmsan.git master
kernel config: https://syzkaller.appspot.com/x/.config?x=46a956fc7a887c60
dashboard link: https://syzkaller.appspot.com/bug?extid=f44badb06036334e867a
compiler: clang version 14.0.0 (/usr/local/google/src/llvm-git-monorepo 2b554920f11c8b763cd9ed9003f4e19b919b8e1f), GNU ld (GNU Binutils for Debian) 2.35.2
patch: https://syzkaller.appspot.com/x/patch.diff?x=163c52a5b00000
Note: testing is done by a robot and is best-effort only.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-19 1:03 ` syzbot
@ 2021-12-19 17:17 ` Andrew Lunn
2021-12-19 21:15 ` Pavel Skripkin
0 siblings, 1 reply; 7+ messages in thread
From: Andrew Lunn @ 2021-12-19 17:17 UTC (permalink / raw)
To: syzbot
Cc: davem, glider, kuba, linux-kernel, linux-usb, linux, netdev,
paskripkin, syzkaller-bugs
On Sat, Dec 18, 2021 at 05:03:09PM -0800, syzbot wrote:
> Hello,
>
> syzbot has tested the proposed patch and the reproducer did not trigger any issue:
>
> Reported-and-tested-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
So it looks like you were right. But maybe ret < sizeof(smsr) would be
better?
Andrew
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [syzbot] KMSAN: uninit-value in asix_mdio_read (2)
2021-12-19 17:17 ` Andrew Lunn
@ 2021-12-19 21:15 ` Pavel Skripkin
0 siblings, 0 replies; 7+ messages in thread
From: Pavel Skripkin @ 2021-12-19 21:15 UTC (permalink / raw)
To: Andrew Lunn, syzbot
Cc: davem, glider, kuba, linux-kernel, linux-usb, linux, netdev,
syzkaller-bugs
On 12/19/21 20:17, Andrew Lunn wrote:
> On Sat, Dec 18, 2021 at 05:03:09PM -0800, syzbot wrote:
>> Hello,
>>
>> syzbot has tested the proposed patch and the reproducer did not trigger any issue:
>>
>> Reported-and-tested-by: syzbot+f44badb06036334e867a@syzkaller.appspotmail.com
>
> So it looks like you were right. But maybe ret < sizeof(smsr) would be
> better?
>
I agree, this looks much cleaner.
Also, the bug you mentioned in previous email is also real, so I will
prepare a patch series if you haven't fixed it by yourself :)
With regards,
Pavel Skripkin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2021-12-19 21:15 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-18 11:07 [syzbot] KMSAN: uninit-value in asix_mdio_read (2) syzbot
2021-12-18 21:14 ` Pavel Skripkin
2021-12-18 21:42 ` Andrew Lunn
2021-12-18 22:01 ` Pavel Skripkin
2021-12-19 1:03 ` syzbot
2021-12-19 17:17 ` Andrew Lunn
2021-12-19 21:15 ` Pavel Skripkin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).