Commit 6ab535a7 authored by Marcel Holtmann's avatar Marcel Holtmann

Bluetooth: Don't use non-resolvable private address for passive scanning

The usage of non-resovlable private addresses for passive scanning is
a bad idea. Passive scanning will not send any SCAN_REQ and thus using
your identity address for passive scanning is not a privacy issue.

It is important to use the identity address during passive scanning
since that is the only way devices using direct advertising will be
reported correctly by the controller. This is overlooked detail in
the Bluetooth specification that current controllers are not able
to report direct advertising events for other than their current
address.

When remote peers are using direct advertising and scanning is done
with non-resolvable private address these devices will not be found.
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
Signed-off-by: default avatarJohan Hedberg <johan.hedberg@intel.com>
parent fee746b0
...@@ -5250,12 +5250,13 @@ void hci_req_add_le_passive_scan(struct hci_request *req) ...@@ -5250,12 +5250,13 @@ void hci_req_add_le_passive_scan(struct hci_request *req)
struct hci_dev *hdev = req->hdev; struct hci_dev *hdev = req->hdev;
u8 own_addr_type; u8 own_addr_type;
/* Set require_privacy to true to avoid identification from /* Set require_privacy to false since no SCAN_REQ are send
* unknown peer devices. Since this is passive scanning, no * during passive scanning. Not using an unresolvable address
* SCAN_REQ using the local identity should be sent. Mandating * here is important so that peer devices using direct
* privacy is just an extra precaution. * advertising with our address will be correctly reported
* by the controller.
*/ */
if (hci_update_random_address(req, true, &own_addr_type)) if (hci_update_random_address(req, false, &own_addr_type))
return; return;
memset(&param_cp, 0, sizeof(param_cp)); memset(&param_cp, 0, sizeof(param_cp));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment