Bluetooth: Fix dereferencing conn variable before NULL check
authorJohan Hedberg <johan.hedberg@intel.com>
Sat, 6 Sep 2014 03:59:10 +0000 (06:59 +0300)
committerMarcel Holtmann <marcel@holtmann.org>
Mon, 8 Sep 2014 17:07:57 +0000 (19:07 +0200)
This patch fixes the following type of static analyzer warning (and
probably a real bug as well as the NULL check should be there for a
reason):

net/bluetooth/smp.c:1182 smp_conn_security() warn: variable dereferenced before check 'conn' (see line 1174)

Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
net/bluetooth/smp.c

index 1201670afe381ac7e637137963dfd78e708623bb..560f78a9f960a59b64330c2618539ba29cf3d8ea 100644 (file)
@@ -1210,7 +1210,7 @@ static u8 smp_cmd_security_req(struct l2cap_conn *conn, struct sk_buff *skb)
 int smp_conn_security(struct hci_conn *hcon, __u8 sec_level)
 {
        struct l2cap_conn *conn = hcon->l2cap_data;
-       struct l2cap_chan *chan = conn->smp;
+       struct l2cap_chan *chan;
        struct smp_chan *smp;
        __u8 authreq;
        int ret;
@@ -1221,6 +1221,8 @@ int smp_conn_security(struct hci_conn *hcon, __u8 sec_level)
        if (!conn)
                return 1;
 
+       chan = conn->smp;
+
        if (!test_bit(HCI_LE_ENABLED, &hcon->hdev->dev_flags))
                return 1;