netdev
[Top] [All Lists]

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

To: Thomas Graf <tgraf@xxxxxxx>
Subject: Re: memory leak in net/sched/ipt.c?
From: jamal <hadi@xxxxxxxxxx>
Date: 23 Mar 2005 08:32:40 -0500
Cc: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>, Yichen Xie <yxie@xxxxxxxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>, kaber@xxxxxxxxx
In-reply-to: <20050323132332.GQ3086@xxxxxxxxxxxxxx>
Organization: jamalopolous
References: <E1DE44X-0001QM-00@xxxxxxxxxxxxxxxxxxxxxxxx> <1111581618.1088.72.camel@xxxxxxxxxxxxxxxx> <20050323125516.GP3086@xxxxxxxxxxxxxx> <1111583497.1089.92.camel@xxxxxxxxxxxxxxxx> <20050323132332.GQ3086@xxxxxxxxxxxxxx>
Reply-to: hadi@xxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
On Wed, 2005-03-23 at 08:23, Thomas Graf wrote:

> 
> kfree simply does nothing if it is given a null pointer so that
> goto rtattr_failure for t == NULL  is handled just fine without
> a check. I will never get used to this behaviour and policy as
> well though, it somewhat makes code less readable.
> 

I cant get used to it either; actually i dont think i would be able 
to stop my fingers ;-> 

> > didnt understand the janitor part.
> 
> It will probably be removed again by one of the regular 'remove
> unnecessary pre kfree checks' patchsets.
> 

Yes, already Jan Engelhardt <jengelh@xxxxxxxxxxxxxxx> has made that
point - although he did not post to netdev!! ;->

So ignore my comment Herbert

cheers,
jamal


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