diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-11-14 22:00:27 +0100 |
|---|---|---|
| committer | Marcel Holtmann <marcel@holtmann.org> | 2015-11-19 17:50:32 +0100 |
| commit | 06fbb3d5c7ff366fe7ab7b4157bdb3096fca6d09 (patch) | |
| tree | acc069f2c4ec277cb4024e8f7a9f897ce959be7b /net/bluetooth/cmtp | |
| parent | mac802154: Delete an unnecessary check before the function call "kfree_skb" (diff) | |
| download | linux-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/cmtp')
| -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 * 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; } |