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



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.

johannes

Attachment: signature.asc
Description: This is a digitally signed message part



Relevant Pages

  • Re: [git pull] drm fixes
    ... Speaking of which, what about leak fixes? ... The third jffs2 patch in that tree is a fix for ctime semantics which is ... Not strictly a regression, as it's a new driver in 2.6.35 -- ...
    (Linux-Kernel)
  • Re: Possible memory leak in net/wireless/scan.c
    ... I'll try this later today to see if it fixes the leak. ... I can now confirm that it fixes the memory leak. ... More majordomo info at http://vger.kernel.org/majordomo-info.html ...
    (Linux-Kernel)
  • Re: string memory leak?
    ... have finally tracked down the source by using the Microsoft tool umdh ... I seem to be unable to reproduce the problem in ... I still have one more leak that has something to do with std::string. ... I am surprised that the dinkumware fixes did not address this leak as ...
    (microsoft.public.vc.stl)
  • [Patch 2.6] dm-crypt: zero key before freeing it
    ... dm-crypt does not clear struct crypt_config before freeing it. ... information on the key could leak f.e. ... to a swsusp image even after the ... fixes it. ...
    (Linux-Kernel)
  • Re: Possible memory leak in net/wireless/scan.c
    ... I'll try this later today to see if it fixes the leak. ... I have already sent that exact patch. ...
    (Linux-Kernel)