[PATCH BlueZ 2/2] bearer: Fix LE Connect and Disconnect ignoring auto-connect

[Kevin Dewald <kevin@simpleble.org>]

Bearer.LE1.Disconnect is documented to disable incoming connections of
a device that is not trusted until Connect is called again, as
Device1.Disconnect does, but it left the device in the auto-connect
list. A device that keeps advertising, such as a HID device, was then
connected back right after the disconnection.

Bearer.LE1.Connect did not put a device back in the auto-connect list
either, unlike Device1.Connect. A device taken out of it by
Device1.Disconnect, or left out of it because it was temporary when its
profiles were probed, was then not reconnected after losing the
connection.

Fix it by pausing and resuming auto-connect as Device1 does.

Fixes: 4e21ebcaa5d0 ("bearer: Implement Connect/Disconnect methods")
Assisted-by: Claude:claude-opus-5.5
---
 src/bearer.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/src/bearer.c b/src/bearer.c
index 5285e0274..e11a844d8 100644
--- a/src/bearer.c
+++ b/src/bearer.c
@@ -153,6 +153,7 @@ static DBusMessage *bearer_connect(DBusConnection *conn, DBusMessage *msg,
 								msg, NULL);
 	else {
 		btd_device_set_temporary(device, false);
+		device_resume_auto_connect(device);
 		err = device_connect_le(device);
 		if (err < 0)
 			return btd_error_failed(msg, strerror(-err));
@@ -183,6 +184,10 @@ static DBusMessage *bearer_disconnect(DBusConnection *conn, DBusMessage *msg,
 		return NULL;
 	}
 
+	/* Auto-connect is only done over LE */
+	if (bearer->type != BDADDR_BREDR)
+		device_pause_auto_connect(device);
+
 	if (msg)
 		queue_push_tail(bearer->disconnects, dbus_message_ref(msg));
 
-- 
2.54.0 (Apple Git-157)