[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <690bc7d8.050a0220.baf87.0073.GAE@google.com>
Date: Wed, 05 Nov 2025 13:55:36 -0800
From: syzbot <syzbot@...kaller.appspotmail.com>
To: jacob.e.keller@...el.com
Cc: davem@...emloft.net, dharanitharan725@...il.com, edumazet@...gle.com,
gregkh@...uxfoundation.org, jacob.e.keller@...el.com, kuba@...nel.org,
linux-usb@...r.kernel.org, netdev@...r.kernel.org, pabeni@...hat.com
Subject: Re: [PATCH v2] usb: rtl8150: Initialize buffers to fix KMSAN
uninit-value in rtl8150_open
>
>
> On 11/5/2025 11:47 AM, Dharanitharan R wrote:
>> diff --git a/drivers/net/usb/rtl8150.c b/drivers/net/usb/rtl8150.c
>> index f1a868f0032e..a7116d03c3d3 100644
>> --- a/drivers/net/usb/rtl8150.c
>> +++ b/drivers/net/usb/rtl8150.c
>> @@ -735,33 +735,30 @@ static int rtl8150_open(struct net_device *netdev)
>> rtl8150_t *dev = netdev_priv(netdev);
>> int res;
>>
>> - if (dev->rx_skb == NULL)
>> - dev->rx_skb = pull_skb(dev);
>> - if (!dev->rx_skb)
>> - return -ENOMEM;
>> -
>
> None of the changes in the diff make any sense, as you remove the only
> place where rx_skb is initialized in the first place.
>
>> set_registers(dev, IDR, 6, netdev->dev_addr);
>>
>> /* Fix: initialize memory before using it (KMSAN uninit-value) */
>> memset(dev->rx_skb->data, 0, RTL8150_MTU);
>> memset(dev->intr_buff, 0, INTBUFSIZE);
>>
>
> This isn't even in the current driver code, but its shown as part of the
> diff context. Based on your commit description this is probably what
> you're trying to insert? But its obviously not a properly formatted or
> generated patch. It reeks of being generated by a bad LLM.
>
> Please don't waste reviewers time with this kind of generated nonsense.
I see the command but can't find the corresponding bug.
The email is sent to syzbot+HASH@...kaller.appspotmail.com address
but the HASH does not correspond to any known bug.
Please double check the address.
Powered by blists - more mailing lists