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 3163330..b9aa986 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)