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

Bluetooth: Fix wrong identity address during connection failures 



When the connection attempt fails, the address information are not
provided in the HCI_LE_Connection_Complete event. So use the original
information from the connection to reconstruct the identity address.

This is important when a connection attempt has been made using the
identity address, but the cached resolvable random address has changed
in the meantime. The failure event needs to use the identity address
and not the resolvable random address.

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

net/bluetooth/hci_conn.c

+13 −0
Original line number Diff line number Diff line
@@ -623,6 +623,19 @@ static struct hci_conn *hci_connect_le(struct hci_dev *hdev, bdaddr_t *dst, 
	else

		dst_type = ADDR_LE_DEV_RANDOM;



	/* When given an identity address with existing identity

	 * resolving key, the connection needs to be established

	 * to a resolvable random address.

	 *

	 * This uses the cached random resolvable address from

	 * a previous scan. When no cached address is available,

	 * try connecting to the identity address instead.

	 *

	 * Storing the resolvable random address is required here

	 * to handle connection failures. The address will later

	 * be resolved back into the original identity address

	 * from the connect request.

	 */

	irk = hci_find_irk_by_addr(hdev, dst, dst_type);

	if (irk && bacmp(&irk->rpa, BDADDR_ANY)) {

		dst = &irk->rpa;

net/bluetooth/hci_event.c

+10 −2
Original line number Diff line number Diff line
@@ -3601,8 +3601,16 @@ static void hci_le_conn_complete_evt(struct hci_dev *hdev, struct sk_buff *skb) 
		}

	}



	/* Track the connection based on the Identity Address from now on */

	irk = hci_get_irk(hdev, &ev->bdaddr, ev->bdaddr_type);

	/* Lookup the identity address from the stored connection

	 * address and address type.

	 *

	 * When establishing connections to an identity address, the

	 * connection procedure will store the resolvable random

	 * address first. Now if it can be converted back into the

	 * identity address, start using the identity address from

	 * now on.

	 */

	irk = hci_get_irk(hdev, &conn->dst, conn->dst_type);

	if (irk) {

		bacpy(&conn->dst, &irk->bdaddr);

		conn->dst_type = irk->addr_type;

CRA Git | Maintained and supported by SUSTech CRA and CCSE