linux-kernel - Re: KMSAN: uninit-value in mii_nway

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <000000000000b90a470621858e8b@google.com>
Date: Sat, 07 Sep 2024 04:09:29 -0700
From: syzbot <syzbot+1f53a30781af65d2c955@...kaller.appspotmail.com>
To: hridesh699@...il.com
Cc: hridesh699@...il.com, linux-kernel@...r.kernel.org, 
	syzkaller-bugs@...glegroups.com
Subject: Re: KMSAN: uninit-value in mii_nway_restart

> #syz test

This bug is already marked as fixed. No point in testing.

> diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c
> index 48d7d278631e..2e2bb22e60ea 100644
> --- a/drivers/net/usb/dm9601.c
> +++ b/drivers/net/usb/dm9601.c
> @@ -10,6 +10,7 @@
>
>  //#define DEBUG
>
> +#include "net/net_debug.h"
>  #include <linux/module.h>
>  #include <linux/sched.h>
>  #include <linux/stddef.h>
> @@ -222,13 +223,18 @@ static int dm9601_mdio_read(struct net_device
> *netdev, int phy_id, int loc)
>     struct usbnet *dev = netdev_priv(netdev);
>
>     __le16 res;
> +   int err;
>
>     if (phy_id) {
>         netdev_dbg(dev->net, "Only internal phy supported\n");
>         return 0;
>     }
>
> -   dm_read_shared_word(dev, 1, loc, &res);
> +   err = dm_read_shared_word(dev, 1, loc, &res);
> +   if (err < 0) {
> +       netdev_err(dev->net, "MDIO read error: %d\n", err);
> +       return err;
> +   }
>
>     netdev_dbg(dev->net,
>            "dm9601_mdio_read() phy_id=0x%02x, loc=0x%02x, returns=0x%04x\n",