| 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@postel.suug.ch> |
| Organization: | jamalopolous |
| References: | <E1DE44X-0001QM-00@gondolin.me.apana.org.au> <1111581618.1088.72.camel@jzny.localdomain> <20050323125516.GP3086@postel.suug.ch> <1111583497.1089.92.camel@jzny.localdomain> <20050323132332.GQ3086@postel.suug.ch> |
| 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> |
|---|---|---|
| ||
| Previous by Date: | Re: memory leak in net/sched/ipt.c?, Thomas Graf |
|---|---|
| Next by Date: | Re: memory leak in net/sched/ipt.c?, Jan Engelhardt |
| Previous by Thread: | Re: memory leak in net/sched/ipt.c?, Thomas Graf |
| Next by Thread: | Re: memory leak in net/sched/ipt.c?, Jan Engelhardt |
| Indexes: | [Date] [Thread] [Top] [All Lists] |