From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH] fix memory leak in net/ipv4/tcp_probe.c::tcpprobe_read() Date: Sun, 30 Jul 2006 14:29:22 -0700 Message-ID: <20060730142922.7c1d4f7d@localhost.localdomain> References: <200607302138.02855.jesper.juhl@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Alexey Kuznetsov , James Morris , Hideaki YOSHIFUJI , Pekka Savola , Patrick McHardy , netdev@vger.kernel.org Return-path: Received: from smtp.osdl.org ([65.172.181.4]:8867 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S932348AbWG3Vat (ORCPT ); Sun, 30 Jul 2006 17:30:49 -0400 To: Jesper Juhl , "David S. Miller" In-Reply-To: <200607302138.02855.jesper.juhl@gmail.com> Sender: netdev-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On Sun, 30 Jul 2006 21:38:02 +0200 Jesper Juhl wrote: > There's an obvious memory leak in net/ipv4/tcp_probe.c::tcpprobe_read() > We are not freeing 'tbuf' on error. > Patch below fixes that. > > > Signed-off-by: Jesper Juhl Agreed, thanks for catching it. The whole kfifo interface is kind of annoying have to do an extra copy.