Revert "Bluetooth: Always request for user confirmation for Just Works"
authorPhil Elwell <phil@raspberrypi.com>
Mon, 1 Mar 2021 09:14:35 +0000 (09:14 +0000)
committerDom Cobley <popcornmix@gmail.com>
Mon, 19 Feb 2024 11:31:30 +0000 (11:31 +0000)
This reverts commit 92516cd97fd4d8ad5b1421a0d51771044f453a5f.

Thi commit "Bluetooth: Always request for user confirmation for Just
Works" prevents BLE devices pairing in (at least) the Raspberry Pi OS
GUI. After reverting it, pairing works again.

If another solution to the problem is found then this reversion will
be removed.

See: https://github.com/raspberrypi/linux/issues/4139

Signed-off-by: Phil Elwell <phil@raspberrypi.com>
net/bluetooth/smp.c

index 2882b32..a408084 100644 (file)
@@ -885,16 +885,9 @@ static int tk_request(struct l2cap_conn *conn, u8 remote_oob, u8 auth,
            hcon->io_capability == HCI_IO_NO_INPUT_OUTPUT)
                smp->method = JUST_WORKS;
 
-       /* If Just Works, Continue with Zero TK and ask user-space for
-        * confirmation */
+       /* If Just Works, Continue with Zero TK */
        if (smp->method == JUST_WORKS) {
-               ret = mgmt_user_confirm_request(hcon->hdev, &hcon->dst,
-                                               hcon->type,
-                                               hcon->dst_type,
-                                               passkey, 1);
-               if (ret)
-                       return ret;
-               set_bit(SMP_FLAG_WAIT_USER, &smp->flags);
+               set_bit(SMP_FLAG_TK_VALID, &smp->flags);
                return 0;
        }