Bluetooth: ISO: Fix BIS connection dst_type handling

Vudentz · Vudentz · commit f0c200a4a537 · 2025-10-24T10:20:34.000-04:00
Socket dst_type cannot be directly assigned to hci_conn->type since there domain is different which may lead to the wrong address type being used. Fixes: 6a5ad25 ("Bluetooth: ISO: Fix possible circular locking dependency") Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
diff --git a/net/bluetooth/iso.c b/net/bluetooth/iso.c
@@ -2032,7 +2032,7 @@ static void iso_conn_ready(struct iso_conn *conn)
 		 */
 		if (!bacmp(&hcon->dst, BDADDR_ANY)) {
 			bacpy(&hcon->dst, &iso_pi(parent)->dst);
-			hcon->dst_type = iso_pi(parent)->dst_type;
+			hcon->dst_type = le_addr_type(iso_pi(parent)->dst_type);
 		}
 
 		if (test_bit(HCI_CONN_PA_SYNC, &hcon->flags)) {

Original file line number	Diff line number	Diff line change
`@@ -2032,7 +2032,7 @@ static void iso_conn_ready(struct iso_conn *conn)`
`2032`	`2032`	`*/`
`2033`	`2033`	`if (!bacmp(&hcon->dst, BDADDR_ANY)) {`
`2034`	`2034`	`bacpy(&hcon->dst, &iso_pi(parent)->dst);`
`2035`		`- hcon->dst_type = iso_pi(parent)->dst_type;`
	`2035`	`+ hcon->dst_type = le_addr_type(iso_pi(parent)->dst_type);`
`2036`	`2036`	`}`
`2037`	`2037`
`2038`	`2038`	`if (test_bit(HCI_CONN_PA_SYNC, &hcon->flags)) {`