Bluetooth: mgmt: Fix refresh cached connection info

Set the connection data before calling get_conn_info_sync, so it can be
verified the connection is still connected, before refreshing cached
values.

Fixes: 47db6b42991e6 ("Bluetooth: hci_sync: Convert MGMT_OP_GET_CONN_INFO")
Signed-off-by: Zhengping Jiang <jiangzp@google.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>

diff --git a/net/bluetooth/mgmt.c b/net/bluetooth/mgmt.c
index ed8cd4a81ed36a36418cc0d7559eed055d3708aa..f3e4e2c9ec7a70c3c1c1827f80f9e6938d90cae9 100644 (file)
--- a/net/bluetooth/mgmt.c
+++ b/net/bluetooth/mgmt.c
@@ -6821,11 +6821,14 @@ static int get_conn_info(struct sock *sk, struct hci_dev *hdev, void *data,
 
                cmd = mgmt_pending_new(sk, MGMT_OP_GET_CONN_INFO, hdev, data,
                                       len);
-               if (!cmd)
+               if (!cmd) {
                        err = -ENOMEM;
-               else
+               } else {
+                       hci_conn_hold(conn);
+                       cmd->user_data = hci_conn_get(conn);
                        err = hci_cmd_sync_queue(hdev, get_conn_info_sync,
                                                 cmd, get_conn_info_complete);
+               }
 
                if (err < 0) {
                        mgmt_cmd_complete(sk, hdev->id, MGMT_OP_GET_CONN_INFO,
@@ -6837,9 +6840,6 @@ static int get_conn_info(struct sock *sk, struct hci_dev *hdev, void *data,
                        goto unlock;
                }
 
-               hci_conn_hold(conn);
-               cmd->user_data = hci_conn_get(conn);
-
                conn->conn_info_timestamp = jiffies;
        } else {
                /* Cache is valid, just reply with values cached in hci_conn */