Return-Path: From: Syam Sidhardhan To: linux-bluetooth@vger.kernel.org Subject: [PATCH 2/2] Bluetooth: Fix ACL alive for long in case of non pariable devices Date: Fri, 21 Dec 2012 19:14:29 +0530 Message-id: <1356097469-24073-2-git-send-email-s.syam@samsung.com> In-reply-to: <1356097469-24073-1-git-send-email-s.syam@samsung.com> References: <1356097469-24073-1-git-send-email-s.syam@samsung.com> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: For certain devices (ex: HID mouse), support for authentication, pairing and bonding is optional. For such devices, the ACL alive for too long after the l2cap disconnection. To avoid keep ACL alive for too long, set the ACL timeout back to HCI_DISCONN_TIMEOUT when l2cap is connected. commit id:a9ea3ed9b71cc3271dd59e76f65748adcaa76422 might have introduce this issue. Signed-off-by: Sang-Ki Park Signed-off-by: Syam Sidhardhan --- I'm not sure whether we need hci_conn_hold() and hci_conn_put() across while updating the disc_timeout. In certain other places in the code it's done. Ex: hci_auth_complete_evt(), hci_link_key_notify_evt() etc. Here I took that as the reference. net/bluetooth/l2cap_core.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c index 82a3bdc..7a544c2 100644 --- a/net/bluetooth/l2cap_core.c +++ b/net/bluetooth/l2cap_core.c @@ -1360,7 +1360,6 @@ static void l2cap_le_conn_ready(struct l2cap_conn *conn) sk = chan->sk; hci_conn_hold(conn->hcon); - conn->hcon->disc_timeout = HCI_DISCONN_TIMEOUT; bacpy(&bt_sk(sk)->src, conn->src); bacpy(&bt_sk(sk)->dst, conn->dst); @@ -1380,6 +1379,10 @@ static void l2cap_conn_ready(struct l2cap_conn *conn) BT_DBG("conn %p", conn); + hci_conn_hold(conn->hcon); + conn->hcon->disc_timeout = HCI_DISCONN_TIMEOUT; + hci_conn_put(conn->hcon); + if (!hcon->out && hcon->type == LE_LINK) l2cap_le_conn_ready(conn); -- 1.7.9.5