Thread (4 messages) 4 messages, 4 authors, 2012-03-01

Re: [patch] Bluetooth: use kfree_skb() instead of kfree()

From: Johan Hedberg <hidden>
Date: 2012-03-01 01:39:49
Also in: kernel-janitors

Hi Dan,

On Tue, Feb 28, 2012, Dan Carpenter wrote:
quoted hunk ↗ jump to hunk
sk_buffs should be freed using kfree_skb().

Signed-off-by: Dan Carpenter <redacted>
diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c
index bdcfbf0..df3be69 100644
--- a/net/bluetooth/l2cap_core.c
+++ b/net/bluetooth/l2cap_core.c
@@ -4284,7 +4284,7 @@ static inline int l2cap_data_channel(struct l2cap_conn *conn, u16 cid, struct sk
 	if (!chan) {
 		BT_DBG("unknown cid 0x%4.4x", cid);
 		/* Drop packet and return */
-		kfree(skb);
+		kfree_skb(skb);
 		return 0;
 	}
Applied to my bluetooth-next tree. Thanks.

Johan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help