Bluetooth: Fix L2CAP security check
authorLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
Mon, 13 Jun 2011 12:37:35 +0000 (15:37 +0300)
committerGustavo F. Padovan <padovan@profusion.mobi>
Mon, 13 Jun 2011 17:28:52 +0000 (14:28 -0300)
With older userspace versions (using hciops) it might not have the
key type to check if the key has sufficient security for any security
level so it is necessary to check the return of hci_conn_auth to make
sure the connection is authenticated

Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Acked-by: Johan Hedberg <johan.hedberg@intel.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
net/bluetooth/hci_conn.c

index 3163330cd4f1388e3c0b5d593b5e54ca75ba0c12..b9aa9862d14bebe484d827016d5eab1900e0889a 100644 (file)
@@ -611,8 +611,8 @@ auth:
        if (test_and_set_bit(HCI_CONN_ENCRYPT_PEND, &conn->pend))
                return 0;
 
-       hci_conn_auth(conn, sec_level, auth_type);
-       return 0;
+       if (!hci_conn_auth(conn, sec_level, auth_type))
+               return 0;
 
 encrypt:
        if (conn->link_mode & HCI_LM_ENCRYPT)