On Mon, 2024-02-12 at 15:22 -0800, Ben Greear wrote: > I'm seeing a crash due to accessing an 'sta' object in this method that is '-2' as integer. > > It fails the initial check for null STA, but I'm thinking it might should check for IS_ERR(sta) > as well. > > (I have my own patch that references sta before the IS_ERR check later in the code, and this > causes the crash I'm seeing. I guess upstream will not crash in this situation.). Indeed. > My question: Is the patch below a preferred approach, or should I add special checks to where I > access sta and only exit the method lower where it already has the IS_ERR(sta) check? You can do whatever you want in your tree, but I guess generally I'd advocate you assume that the code does what it should ;-) In this case, ERR_PTR(-ENOENT) is used to indicate the station is being deleted, but has not yet been fully removed, and so indeed we still want to reclaim the frames here correctly, which the code does. The comment below even kind of explains that? johannes
