Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
From:   Sonny Sasaka <sonnysasaka@chromium.org>
Date:   Fri, 27 Mar 2020 17:12:04 -0700
Message-ID: <CAOxioNkrxDEKuX4AeHWkDSHrwmsBqYXKRi-UtyUqfs1Rw7pePg@mail.gmail.com>
Subject: [PATCH] core/device: Handle Just-Works auto-accept
To:     BlueZ <linux-bluetooth@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

The kernel starts to always request confirmation to BlueZ daemon for
Just-Works pairing. In this patch the daemon does auto-accept if the
client has clearly indicated a pairing intent by calling the Pair()
D-Bus API.
---
 src/device.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/src/device.c b/src/device.c
index 69f98e488..db14cc129 100644
--- a/src/device.c
+++ b/src/device.c
@@ -6168,12 +6168,22 @@ int device_confirm_passkey(struct btd_device
*device, uint8_t type,

  auth->passkey = passkey;

- if (confirm_hint)
+ if (confirm_hint) {
+ if (device->bonding != NULL) {
+ /* We know the client has indicated the intent to pair
+ * with the peer device, so we can auto-accept. */
+ btd_adapter_confirm_reply(device->adapter,
+  &device->bdaddr,
+  type, TRUE);
+ return 0;
+ }
+
  err = agent_request_authorization(auth->agent, device,
  confirm_cb, auth, NULL);
- else
+ } else {
  err = agent_request_confirmation(auth->agent, device, passkey,
  confirm_cb, auth, NULL);
+ }

  if (err < 0) {
  if (err == -EINPROGRESS) {
-- 
2.17.1