* [PATCH 2/4] iw: Avoid possible memory leak for cb. cb got memory from nl_cb_alloc() but not doing free
@ 2015-06-26 9:10 Amit Khatri
2015-08-13 8:52 ` Johannes Berg
0 siblings, 1 reply; 2+ messages in thread
From: Amit Khatri @ 2015-06-26 9:10 UTC (permalink / raw)
To: johannes@sipsolutions.net
Cc: linux-wireless@vger.kernel.org, Rahul Jain, HEMANSHU SRIVASTAVA,
Nitin Jhanwar
SGkgSm9oYW5uZXMsDQoNClN1YmplY3Q6IFtQQVRDSCAyLzRdIGl3OiBBdm9pZCBwb3NzaWJsZSBt
ZW1vcnkgbGVhayBmb3IgY2IuIGNiIGdvdCBtZW1vcnkgZnJvbQ0KIG5sX2NiX2FsbG9jKCkgYnV0
IG5vdCBkb2luZyBmcmVlDQoNClNpZ25lZC1vZmYtYnk6IEFtaXQgS2hhdHJpIDxhbWl0LmtoYXRy
aUBzYW1zdW5nLmNvbT4NClNpZ25lZC1vZmYtYnk6IFJhaHVsIEphaW4gPHJhaHVsLmphaW5Ac2Ft
c3VuZy5jb20+DQotLS0NCiBpdy5jIHwgMiArLQ0KIDEgZmlsZSBjaGFuZ2VkLCAxIGluc2VydGlv
bigrKSwgMSBkZWxldGlvbigtKQ0KDQpkaWZmIC0tZ2l0IGEvaXcuYyBiL2l3LmMNCmluZGV4IGRj
OTk1NjYuLjFlOTEzYjYgMTAwNjQ0DQotLS0gYS9pdy5jDQorKysgYi9pdy5jDQpAQCAtNDc3LDgg
KzQ3Nyw4IEBAIHN0YXRpYyBpbnQgX19oYW5kbGVfY21kKHN0cnVjdCBubDgwMjExX3N0YXRlICpz
dGF0ZSwgZW51bSBpZF9pbnB1dCBpZGJ5LA0KIAl3aGlsZSAoZXJyID4gMCkNCiAJCW5sX3JlY3Zt
c2dzKHN0YXRlLT5ubF9zb2NrLCBjYik7DQogIG91dDoNCi0JbmxfY2JfcHV0KGNiKTsNCiAgb3V0
X2ZyZWVfbXNnOg0KKwlubF9jYl9wdXQoY2IpOw0KIAlubG1zZ19mcmVlKG1zZyk7DQogCXJldHVy
biBlcnI7DQogIG5sYV9wdXRfZmFpbHVyZToNCi0tIA0KMS45LjE=
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH 2/4] iw: Avoid possible memory leak for cb. cb got memory from nl_cb_alloc() but not doing free
2015-06-26 9:10 [PATCH 2/4] iw: Avoid possible memory leak for cb. cb got memory from nl_cb_alloc() but not doing free Amit Khatri
@ 2015-08-13 8:52 ` Johannes Berg
0 siblings, 0 replies; 2+ messages in thread
From: Johannes Berg @ 2015-08-13 8:52 UTC (permalink / raw)
To: amit.khatri
Cc: linux-wireless@vger.kernel.org, Rahul Jain, HEMANSHU SRIVASTAVA,
Nitin Jhanwar
On Fri, 2015-06-26 at 09:10 +0000, Amit Khatri wrote:
> Hi Johannes,
>
> Subject: [PATCH 2/4] iw: Avoid possible memory leak for cb. cb got
> memory from
> nl_cb_alloc() but not doing free
Please fix your submission process to not mangle patches. I've applied
patch 1 in this series and fixed it up manually, but having the subject
again inside the email just makes it harder for me to pick up the
patches.
> Signed-off-by: Amit Khatri <amit.khatri@samsung.com>
> Signed-off-by: Rahul Jain <rahul.jain@samsung.com>
> ---
> iw.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/iw.c b/iw.c
> index dc99566..1e913b6 100644
> --- a/iw.c
> +++ b/iw.c
> @@ -477,8 +477,8 @@ static int __handle_cmd(struct nl80211_state
> *state, enum id_input idby,
> while (err > 0)
> nl_recvmsgs(state->nl_sock, cb);
> out:
> - nl_cb_put(cb);
> out_free_msg:
> + nl_cb_put(cb);
>
Please don't leave two equivalent labels, remove one of them and
replace the users.
johannes
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2015-08-13 8:52 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-26 9:10 [PATCH 2/4] iw: Avoid possible memory leak for cb. cb got memory from nl_cb_alloc() but not doing free Amit Khatri
2015-08-13 8:52 ` Johannes Berg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).