Commit e5469adb authored Jan 05, 2024 by Luiz Augusto von Dentz

Bluetooth: hci_sync: Fix accept_list when attempting to suspend

During suspend, only wakeable devices can be in acceptlist, so if the
device was previously added it needs to be removed otherwise the device
can end up waking up the system prematurely.

Fixes: 3b420553 ("Bluetooth: hci_sync: Fix attempting to suspend with unfiltered passive scan")
Signed-off-by: Clancy Shang <clancy.shang@quectel.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Reviewed-by: Paul Menzel <pmenzel@molgen.mpg.de>

parent 2449007d

net/bluetooth/hci_sync.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -2206,8 +2206,11 @@ static int hci_le_add_accept_list_sync(struct hci_dev *hdev,

		/* During suspend, only wakeable devices can be in acceptlist */
		if (hdev->suspended &&
		!(params->flags & HCI_CONN_FLAG_REMOTE_WAKEUP))
		!(params->flags & HCI_CONN_FLAG_REMOTE_WAKEUP)) {
		hci_le_del_accept_list_sync(hdev, &params->addr,
		params->addr_type);
		return 0;
		}

		/* Select filter policy to accept all advertising */
		if (*num_entries >= hdev->le_accept_list_size)