[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cef19337-5ff3-c0cd-33ef-4f9990bcd4ec@lwfinger.net>
Date: Sun, 1 Aug 2021 15:15:52 -0500
From: Larry Finger <Larry.Finger@...inger.net>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Cc: LKML <linux-kernel@...r.kernel.org>, linux-staging@...ts.linux.dev
Subject: Re: kernel BUG in new r8188eu
On 8/1/21 1:31 AM, Greg Kroah-Hartman wrote:
>
> I am worried that my "remove the wrapper" logic got something wrong
> here, so if you could test the revert of that, I would appreciate it.
>
> I think I need to go buy one of these devices so I can test cleanups
> locally...
The bad commit was 9ff1cc4ef80e ("staging: r8188eu: remove rtw_buf_free()
function").
Looking at it, the only difference between the original wrapper and the new code
is that the wrapper zeros the len variable before the kfree() call, but making
that change did not help. Reverting that patch restores the old behavior.
There are lots of these on Ebay from $2.47 up with free shipping!
Larry
Powered by blists - more mailing lists