Commit ee3c3ca5 authored by Marcel Holtmann's avatar Marcel Holtmann Committed by Johan Hedberg
Browse files

Bluetooth: Clear discovery filter before starting background scan 



Currently the discovery filter information are only cleared when the
actual discovery procedure has been stopped. To make sure that none
of the filters interfere with the background scanning and its device
found event reporting, clear the filter before starting background
scanning.

This means that the discovery filter is now cleared before either
Start Discovery, Start Service Discovery or background scanning.

Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
parent 22078800

net/bluetooth/hci_core.c

+9 −11
Original line number Diff line number Diff line
@@ -2052,17 +2052,6 @@ void hci_discovery_set_state(struct hci_dev *hdev, int state) 
	case DISCOVERY_STOPPED:

		hci_update_background_scan(hdev);



		/* Reset RSSI and UUID filters to ensure Start Discovery

		 * and Start Service Discovery operate properly no matter

		 * which one started the previous discovery.

		 *

		 * While the Start Discovery and Start Service Discovery

		 * operations will set proper values for RSSI and UUID

		 * count, it is important to actually free the allocated

		 * list of UUIDs here.

		 */

		hci_discovery_filter_clear(hdev);



		if (old_state != DISCOVERY_STARTING)

			mgmt_discovering(hdev, 0);

		break;
@@ -5679,6 +5668,15 @@ void hci_update_background_scan(struct hci_dev *hdev) 
	if (hdev->discovery.state != DISCOVERY_STOPPED)

		return;



	/* Reset RSSI and UUID filters when starting background scanning

	 * since these filters are meant for service discovery only.

	 *

	 * The Start Discovery and Start Service Discovery operations

	 * ensure to set proper values for RSSI threshold and UUID

	 * filter list. So it is safe to just reset them here.

	 */

	hci_discovery_filter_clear(hdev);



	hci_req_init(&req, hdev);



	if (list_empty(&hdev->pend_le_conns) &&

CRA Git | Maintained and supported by SUSTech CRA and CCSE