From: Johannes Berg <johannes@sipsolutions.net>
To: Ola Olsson <ola1olsson@gmail.com>
Cc: linux-wireless <linux-wireless@vger.kernel.org>,
Ola Olsson <ola.olsson@sonymobile.com>
Subject: Re: [PATCH] nl80211: Potential memory leaks in reg.c
Date: Tue, 15 Dec 2015 13:33:08 +0100 [thread overview]
Message-ID: <1450182788.3356.14.camel@sipsolutions.net> (raw)
In-Reply-To: <CABAco3CgkoCYdHZV6nBfV+CUpD379XR69EiznXvu42p5UC96gw@mail.gmail.com> (sfid-20151215_131934_996381_B65C896A)
On Tue, 2015-12-15 at 13:19 +0100, Ola Olsson wrote:
> I'm trying to learn how the code works so I'm reading through it line
> by line. I have seen so many memory leaks in my days so I know where
> to look ;)
Ok :)
> They are indeed obscure since we need an error to actually trigger
> them but I thought I might as well share the info when I found them.
Of course, the patches are very welcome!
I'm just surprised that none of the tools we (and others) typically run
found these, and thought perhaps you had a better tool than we do...
I guess it's called "brain" and we just don't use it right ;-)
johannes
next prev parent reply other threads:[~2015-12-15 12:33 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-12-13 18:12 [PATCH] nl80211: Potential memory leaks in reg.c Ola Olsson
2015-12-15 12:09 ` Johannes Berg
2015-12-15 12:19 ` Ola Olsson
2015-12-15 12:33 ` Johannes Berg [this message]
2015-12-16 9:14 ` Ola Olsson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1450182788.3356.14.camel@sipsolutions.net \
--to=johannes@sipsolutions.net \
--cc=linux-wireless@vger.kernel.org \
--cc=ola.olsson@sonymobile.com \
--cc=ola1olsson@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.