Bluetooth: Remove unnecessary use of hci_dev_list_lock

The get_connections function has no need to use hci_dev_list_lock. The
code was there probably because of a copy-paste mistake.

Signed-off-by: Johan Hedberg <[email protected]>
Signed-off-by: Gustavo F. Padovan <[email protected]>

diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
index dae382ce70201d5f4adbacde509c665c467d95ac..fe835b8e493a031448499e2afbebaa807755fee4 100644 (file)
--- a/net/bluetooth/mgmt.c
+++ b/net/bluetooth/mgmt.c
@@ -1092,8 +1092,6 @@ static int get_connections(struct sock *sk, u16 index)
 
        put_unaligned_le16(count, &rp->conn_count);
 
-       read_lock(&hci_dev_list_lock);
-
        i = 0;
        list_for_each(p, &hdev->conn_hash.list) {
                struct hci_conn *c = list_entry(p, struct hci_conn, list);
@@ -1101,8 +1099,6 @@ static int get_connections(struct sock *sk, u16 index)
                bacpy(&rp->conn[i++], &c->dst);
        }
 
-       read_unlock(&hci_dev_list_lock);
-
        err = cmd_complete(sk, index, MGMT_OP_GET_CONNECTIONS, rp, rp_len);
 
 unlock: