diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-11-14 16:00:27 -0500 |
---|---|---|
committer | Marcel Holtmann <marcel@holtmann.org> | 2015-11-19 11:50:32 -0500 |
commit | 06fbb3d5c7ff366fe7ab7b4157bdb3096fca6d09 (patch) | |
tree | acc069f2c4ec277cb4024e8f7a9f897ce959be7b /net/bluetooth | |
parent | 56f9ebe641d613916d3dce710004d48ab66660fa (diff) |
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.c | 3 |
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 * | |||
178 | cmtp_add_msgpart(session, id, skb->data + hdrlen, len); | 178 | cmtp_add_msgpart(session, id, skb->data + hdrlen, len); |
179 | break; | 179 | break; |
180 | default: | 180 | default: |
181 | if (session->reassembly[id] != NULL) | 181 | kfree_skb(session->reassembly[id]); |
182 | kfree_skb(session->reassembly[id]); | ||
183 | session->reassembly[id] = NULL; | 182 | session->reassembly[id] = NULL; |
184 | break; | 183 | break; |
185 | } | 184 | } |