Forwarded: [PATCH] Bluetooth: hci_conn: Fix memory leak in tx_q on connection teardown

[syzbot <syzbot+6e76aa21aaf2d8be6034@syzkaller.appspotmail.com>]

For archival purposes, forwarding an incoming command email to
linux-kernel@vger.kernel.org, syzkaller-bugs@googlegroups.com.

***

Subject: [PATCH] Bluetooth: hci_conn: Fix memory leak in tx_q on connection teardown
Author: kartikey406@gmail.com

#syz test: git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master

Cloned SKBs queued in conn->tx_q.queue for TX timestamp tracking are
not freed if the connection is torn down before all completion events
arrive. Add skb_queue_purge() in hci_conn_cleanup() to fix the leak.

Reported-by: syzbot+6e76aa21aaf2d8be6034@syzkaller.appspotmail.com
Closes: https://syzkaller.appspot.com/bug?extid=6e76aa21aaf2d8be6034
Signed-off-by: Deepanshu Kartikey <kartikey406@gmail.com>
---
 net/bluetooth/hci_conn.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
index c3f7828bf9d5..3fb93faf60f3 100644
--- a/net/bluetooth/hci_conn.c
+++ b/net/bluetooth/hci_conn.c
@@ -148,6 +148,7 @@ static void hci_conn_cleanup(struct hci_conn *conn)
 		hci_remove_link_key(hdev, &conn->dst);
 
 	hci_chan_list_flush(conn);
+	skb_queue_purge(&conn->tx_q.queue);
 
 	if (HCI_CONN_HANDLE_UNSET(conn->handle))
 		ida_free(&hdev->unset_handle_ida, conn->handle);
-- 
2.43.0