Bluetooth: Fix SMP security level when we have no IO capabilities
authorJohan Hedberg <johan.hedberg@intel.com>
Thu, 11 Sep 2014 00:37:44 +0000 (17:37 -0700)
committerMarcel Holtmann <marcel@holtmann.org>
Thu, 11 Sep 2014 00:45:24 +0000 (02:45 +0200)
When the local IO capability is NoInputNoOutput any attempt to convert
the remote authentication requirement to a target security level is
futile. This patch makes sure that we set the target security level at
most to MEDIUM if the local IO capability is NoInputNoOutput.

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

index be8371b4eb631620e10deb0f5a5b2740791a9d30..a08b077cb725bd0419c1ed87030efae68eb0c9e0 100644 (file)
@@ -959,7 +959,11 @@ static u8 smp_cmd_pairing_req(struct l2cap_conn *conn, struct sk_buff *skb)
        memcpy(&smp->preq[1], req, sizeof(*req));
        skb_pull(skb, sizeof(*req));
 
-       sec_level = authreq_to_seclevel(auth);
+       if (conn->hcon->io_capability == 0x03)
+               sec_level = BT_SECURITY_MEDIUM;
+       else
+               sec_level = authreq_to_seclevel(auth);
+
        if (sec_level > conn->hcon->pending_sec_level)
                conn->hcon->pending_sec_level = sec_level;
 
@@ -1165,7 +1169,11 @@ static u8 smp_cmd_security_req(struct l2cap_conn *conn, struct sk_buff *skb)
 
        auth = rp->auth_req & AUTH_REQ_MASK;
 
-       sec_level = authreq_to_seclevel(auth);
+       if (hcon->io_capability == 0x03)
+               sec_level = BT_SECURITY_MEDIUM;
+       else
+               sec_level = authreq_to_seclevel(auth);
+
        if (smp_sufficient_security(hcon, sec_level))
                return 0;