Skip to contentSkip to conversation

Bluetooth: hci_qca: Use kfree_skb() instead of kfree()

PATCH In review Dan Carpenter 2 participants 2 messages Aug 15, 2019

Subsystems (3)
Bluetooth drivers Kernel janitors Qualcomm Bluetooth driver
lore
Share & tools
lore

Dan Carpenter

author
#0 lore raw

PATCH 1/1 Bluetooth: hci_qca: Use kfree_skb() instead of kfree()

sk_buff structs need to be freed with kfree_skb().

Fixes 2faa3f15fa2f ("Bluetooth: hci_qca: wcn3990: Drop baudrate change vendor event")
Signed-off-by Dan Carpenter <[email protected]>
---
 drivers/bluetooth/hci_qca.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
drivers/bluetooth/hci_qca.c +1 -1
index a054210ae866..7720e1748262 100644
@@ -893,7 +893,7 @@ static int qca_recv_event(struct hci_dev *hdev, struct sk_buff *skb)
 		if (hdr->evt = HCI_EV_VENDOR)
 			complete(&qca->drop_ev_comp);
 
-		kfree(skb);
+		kfree_skb(skb);
 
 		return 0;
 	}
Signature · 1 line
2.20.1
maintainer
#1 lore raw

Hi Dan,

Quoted 7 lines

sk_buff structs need to be freed with kfree_skb().

Fixes 2faa3f15fa2f ("Bluetooth: hci_qca: wcn3990: Drop baudrate change vendor event")
Signed-off-by Dan Carpenter <[email protected]>

---
drivers/bluetooth/hci_qca.c | 2 +-

1 file changed, 1 insertion(+), 1 deletion(-)

patch has been applied to bluetooth-next tree.

Regards

Marcel