summaryrefslogtreecommitdiffstats
path: root/net/bluetooth
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2015-11-14 22:00:27 +0100
committerMarcel Holtmann <marcel@holtmann.org>2015-11-19 17:50:32 +0100
commit06fbb3d5c7ff366fe7ab7b4157bdb3096fca6d09 (patch)
treeacc069f2c4ec277cb4024e8f7a9f897ce959be7b /net/bluetooth
parentmac802154: Delete an unnecessary check before the function call "kfree_skb" (diff)
downloadlinux-06fbb3d5c7ff366fe7ab7b4157bdb3096fca6d09.tar.xz
linux-06fbb3d5c7ff366fe7ab7b4157bdb3096fca6d09.zip
Bluetooth: Delete an unnecessary check 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 call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Diffstat (limited to 'net/bluetooth')
-rw-r--r--net/bluetooth/cmtp/core.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/net/bluetooth/cmtp/core.c b/net/bluetooth/cmtp/core.c
index 298ed37010e6..9e59b6654126 100644
--- a/net/bluetooth/cmtp/core.c
+++ b/net/bluetooth/cmtp/core.c
@@ -178,8 +178,7 @@ static inline int cmtp_recv_frame(struct cmtp_session *session, struct sk_buff *
cmtp_add_msgpart(session, id, skb->data + hdrlen, len);
break;
default:
- if (session->reassembly[id] != NULL)
- kfree_skb(session->reassembly[id]);
+ kfree_skb(session->reassembly[id]);
session->reassembly[id] = NULL;
break;
}