Re: Memory leak on failed connection attempt

[Tomas Targownik <ttargownik@geicp.com>]

I have isolated the memory leak to the hci_conn struct not being freed
on a hci_conn_del() call if the remote bluetooth device is not present.
The patch below remedies this.

---
net/bluetooth/hci_conn.c |    5 +++++
1 file changed, 5 insertions(+)

diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c
index 7a6f56b..62510f7 100644
--- a/net/bluetooth/hci_conn.c
+++ b/net/bluetooth/hci_conn.c
@@ -375,6 +375,11 @@ int hci_conn_del(struct hci_conn *conn)

hci_dev_put(hdev);

+   if (conn->handle == 0)
+   {
+       kfree(conn);
+   }
+
return 0;
}


Best regards, 
Tomas Targownik