Re: Possible memory leak in net/wireless/scan.c

Johannes Berg
2009-07-07T21:48:29Z

[-- Attachment #1: Type: text/plain, Size: 692 bytes --]

On Tue, 2009-07-07 at 22:29 +0100, Catalin Marinas wrote:

> > > if (found) {
> > > - kref_get(&found->ref);
> > > found->pub.beacon_interval = res->pub.beacon_interval;
> > > found->pub.tsf = res->pub.tsf;
> > > found->pub.signal = res->pub.signal;
> > >
> > > I'll try this later today to see if it fixes the leak. If that's not
> > > correct, I'll post more information about the content of the reported
> > > object (in general, it shouldn't be on any valid list or rb tree since
> > > kmemleak can't find it).
> >
> > I have already sent that exact patch.
>
> OK. I can now confirm that it fixes the memory leak.

Great, thanks for checking.