diff options
author | Jesper Juhl <jesper.juhl@gmail.com> | 2005-11-08 12:41:34 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2005-11-08 12:41:34 -0500 |
commit | a51482bde22f99c63fbbb57d5d46cc666384e379 (patch) | |
tree | 5482ed1c0803edb2ffbd51035de921fb0f72d82b /net/ipv4/ip_output.c | |
parent | ac7c98eca88a854755475fcfe1b2bf5f97f90d99 (diff) |
[NET]: kfree cleanup
From: Jesper Juhl <jesper.juhl@gmail.com>
This is the net/ part of the big kfree cleanup patch.
Remove pointless checks for NULL prior to calling kfree() in net/.
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Arnaldo Carvalho de Melo <acme@conectiva.com.br>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Acked-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Diffstat (limited to 'net/ipv4/ip_output.c')
-rw-r--r-- | net/ipv4/ip_output.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/net/ipv4/ip_output.c b/net/ipv4/ip_output.c index 17758234a3e3..df7f20da422d 100644 --- a/net/ipv4/ip_output.c +++ b/net/ipv4/ip_output.c | |||
@@ -1262,10 +1262,8 @@ int ip_push_pending_frames(struct sock *sk) | |||
1262 | 1262 | ||
1263 | out: | 1263 | out: |
1264 | inet->cork.flags &= ~IPCORK_OPT; | 1264 | inet->cork.flags &= ~IPCORK_OPT; |
1265 | if (inet->cork.opt) { | 1265 | kfree(inet->cork.opt); |
1266 | kfree(inet->cork.opt); | 1266 | inet->cork.opt = NULL; |
1267 | inet->cork.opt = NULL; | ||
1268 | } | ||
1269 | if (inet->cork.rt) { | 1267 | if (inet->cork.rt) { |
1270 | ip_rt_put(inet->cork.rt); | 1268 | ip_rt_put(inet->cork.rt); |
1271 | inet->cork.rt = NULL; | 1269 | inet->cork.rt = NULL; |
@@ -1289,10 +1287,8 @@ void ip_flush_pending_frames(struct sock *sk) | |||
1289 | kfree_skb(skb); | 1287 | kfree_skb(skb); |
1290 | 1288 | ||
1291 | inet->cork.flags &= ~IPCORK_OPT; | 1289 | inet->cork.flags &= ~IPCORK_OPT; |
1292 | if (inet->cork.opt) { | 1290 | kfree(inet->cork.opt); |
1293 | kfree(inet->cork.opt); | 1291 | inet->cork.opt = NULL; |
1294 | inet->cork.opt = NULL; | ||
1295 | } | ||
1296 | if (inet->cork.rt) { | 1292 | if (inet->cork.rt) { |
1297 | ip_rt_put(inet->cork.rt); | 1293 | ip_rt_put(inet->cork.rt); |
1298 | inet->cork.rt = NULL; | 1294 | inet->cork.rt = NULL; |