netdev
[Top] [All Lists]

Re: memory leak in net/sched/ipt.c?

To: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Subject: Re: memory leak in net/sched/ipt.c?
From: jamal <hadi@xxxxxxxxxx>
Date: 23 Mar 2005 07:40:18 -0500
Cc: Yichen Xie <yxie@xxxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>, kaber@xxxxxxxxx
In-reply-to: <E1DE44X-0001QM-00@gondolin.me.apana.org.au>
Organization: jamalopolous
References: <E1DE44X-0001QM-00@gondolin.me.apana.org.au>
Reply-to: hadi@xxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
On Wed, 2005-03-23 at 06:30, Herbert Xu wrote:
> Yichen Xie <yxie@xxxxxxxxxxxxxxx> wrote:
> > Is the memory block allocated on line 315 leaked every time tcp_ipt_dump 
> > is called?
> 
> It seems to be.  This patch should free it.
> 
> Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
> 
> BTW, please report networking bugs to netdev@xxxxxxxxxxxx
> 

Thanks for pointing this out - I _know_ its in the kernel list FAQ.
I was sitting beside R Gooch when he added "thou shalt post to netdev".
And of course i am gonna bitch about it every time someone doesnt
post to netdev;-> 

Just a small correction to patchlet:
The second kfree should check for existence of t.

cheers,
jamal

> Thanks,


<Prev in Thread] Current Thread [Next in Thread>