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

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

From: Andrei Emeltchenko <hidden>
Date: 2012-02-28 07:57:22
Also in: kernel-janitors

On Tue, Feb 28, 2012 at 09:52:46AM +0300, Dan Carpenter wrote:
sk_buffs should be freed using kfree_skb().

Signed-off-by: Dan Carpenter <redacted>
Sorry missed that.

Acked-by: Andrei Emeltchenko <redacted> 
quoted hunk ↗ jump to hunk
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;
 	}
 
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki 
Business Identity Code: 0357606 - 4 
Domiciled in Helsinki 

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help