summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJesper Juhl <jesper.juhl@gmail.com>2006-04-18 14:51:44 -0700
committerDavid S. Miller <davem@sunset.davemloft.net>2006-04-18 15:57:55 -0700
commit63903ca6af3d9424a0c2b176f927fa7e7ab2ae8e (patch)
treeaf0dca981d11cc29b6cb94377bca6b61f1370a0e
parent40daafc80b0f6a950c9252f9f1a242ab5cb6a648 (diff)
downloadlinux-63903ca6af3d9424a0c2b176f927fa7e7ab2ae8e.tar.gz
linux-63903ca6af3d9424a0c2b176f927fa7e7ab2ae8e.tar.bz2
linux-63903ca6af3d9424a0c2b176f927fa7e7ab2ae8e.zip
[NET]: Remove redundant NULL checks before [kv]free
Redundant NULL check before kfree removal from net/ Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com> Acked-by: James Morris <jmorris@namei.org> Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r--net/ipv4/ipcomp.c7
-rw-r--r--net/tipc/name_distr.c3
2 files changed, 3 insertions, 7 deletions
diff --git a/net/ipv4/ipcomp.c b/net/ipv4/ipcomp.c
index 04a429465665..cd810f41af1a 100644
--- a/net/ipv4/ipcomp.c
+++ b/net/ipv4/ipcomp.c
@@ -290,11 +290,8 @@ static void ipcomp_free_scratches(void)
if (!scratches)
return;
- for_each_possible_cpu(i) {
- void *scratch = *per_cpu_ptr(scratches, i);
- if (scratch)
- vfree(scratch);
- }
+ for_each_possible_cpu(i)
+ vfree(*per_cpu_ptr(scratches, i));
free_percpu(scratches);
}
diff --git a/net/tipc/name_distr.c b/net/tipc/name_distr.c
index 953307a9df1d..a3bbc891f959 100644
--- a/net/tipc/name_distr.c
+++ b/net/tipc/name_distr.c
@@ -229,8 +229,7 @@ static void node_is_down(struct publication *publ)
publ->node, publ->ref, publ->key);
assert(p == publ);
write_unlock_bh(&tipc_nametbl_lock);
- if (publ)
- kfree(publ);
+ kfree(publ);
}
/**