Bluetooth: SMP: Fix variable dereferenced before check 'conn'
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Mon, 29 Mar 2021 17:27:04 +0000 (10:27 -0700)
committerMarcel Holtmann <marcel@holtmann.org>
Fri, 2 Apr 2021 09:09:12 +0000 (11:09 +0200)
This fixes kbuild findings:

smatch warnings:
net/bluetooth/smp.c:1633 smp_user_confirm_reply() warn: variable
dereferenced before check 'conn' (see line 1631)

Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/smp.c

index 2def906..5c17acf 100644 (file)
@@ -1628,11 +1628,11 @@ int smp_user_confirm_reply(struct hci_conn *hcon, u16 mgmt_op, __le32 passkey)
        u32 value;
        int err;
 
-       bt_dev_dbg(conn->hcon->hdev, "");
-
        if (!conn)
                return -ENOTCONN;
 
+       bt_dev_dbg(conn->hcon->hdev, "");
+
        chan = conn->smp;
        if (!chan)
                return -ENOTCONN;