From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Marek Lindner Date: Tue, 4 Jun 2013 17:31:35 +0800 References: <1369722202-19422-1-git-send-email-martin@hundeboll.net> <51A450B4.9090901@hundeboll.net> In-Reply-To: <51A450B4.9090901@hundeboll.net> MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Message-Id: <201306041731.35269.lindner_marek@yahoo.de> Subject: Re: [B.A.T.M.A.N.] [PATCH] batman-adv: fix possible memleak in fragmentation Reply-To: The list for a Better Approach To Mobile Ad-hoc Networking List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: b.a.t.m.a.n@lists.open-mesh.org Cc: Martin =?utf-8?q?Hundeb=C3=B8ll?= On Tuesday, May 28, 2013 14:37:40 Martin Hundeb=C3=B8ll wrote: > On 2013-05-28 08:23, Martin Hundeb=C3=B8ll wrote: > > Use a variable for return value in batadv_frag_insert_packet() to free > > the allocated struct batadv_frag_list_entry in all cases where it is not > > inserted into a chain. >=20 > The possible memleak was introduced by=20 > 9b3eab61754d74a93c9840c296013fe3b4a1b606 ("batman-adv: Receive=20 > fragmented packets and merge") Applied in revision 0b415bb. Thanks, Marek