From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Marek Lindner Date: Fri, 17 Jun 2011 07:55:30 +0200 References: <1308220627-27164-1-git-send-email-lindner_marek@yahoo.de> <20110616132149.GB6077@ritirata.org> In-Reply-To: <20110616132149.GB6077@ritirata.org> MIME-Version: 1.0 Content-Type: Text/Plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <201106170755.30634.lindner_marek@yahoo.de> Subject: Re: [B.A.T.M.A.N.] [PATCH] batman-adv: fix cleanup in send_tt_request() 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: The list for a Better Approach To Mobile Ad-hoc Networking On Thursday, June 16, 2011 03:21:49 PM Antonio Quartulli wrote: > On gio, giu 16, 2011 at 12:37:07 +0200, Marek Lindner wrote: > > The ret variable was never set to any value other than 0, therefore > > the skb and tt_req_node would not be freed in case of an error. > > Furthermore, the code assumed tt_req_node being always a valid > > pointer by dereferencing it in the cleanup part. > > > > > > > > Signed-off-by: Marek Lindner > > Acked-by: Antonio Quartulli Applied in revision 74db080. Thanks, Marek