summaryrefslogtreecommitdiff
path: root/net/batman-adv/main.c
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-11-15 08:04:43 +0100
committerAntonio Quartulli <a@unstable.cc>2016-01-09 20:56:00 +0800
commitc799443ee13ef37221732839f1cca6f11c798b7a (patch)
treef4e271b61ae47e0b9ba8d238abcd5e57d300eb22 /net/batman-adv/main.c
parentd737ccbed3e62dd45d631cf69183de005144d05b (diff)
batman-adv: Delete unnecessary checks before the function call "kfree_skb"
The kfree_skb() function tests whether its argument is NULL and then returns immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Marek Lindner <mareklindner@neomailbox.ch> Signed-off-by: Antonio Quartulli <a@unstable.cc>
Diffstat (limited to 'net/batman-adv/main.c')
-rw-r--r--net/batman-adv/main.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/batman-adv/main.c b/net/batman-adv/main.c
index 95fd418e9567..5b678f3471fc 100644
--- a/net/batman-adv/main.c
+++ b/net/batman-adv/main.c
@@ -1184,7 +1184,7 @@ void batadv_tvlv_unicast_send(struct batadv_priv *bat_priv, u8 *src,
ret = true;
out:
- if (skb && !ret)
+ if (!ret)
kfree_skb(skb);
if (orig_node)
batadv_orig_node_free_ref(orig_node);