Bluetooth: ISO: Fix another instance of dst_type handling

[ Upstream commit c403da5e98b04a2aec9cfb25cbeeb28d7ce29975 ]

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: 6a5ad251b7cd ("Bluetooth: ISO: Fix possible circular locking dependency")
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>

diff --git a/net/bluetooth/iso.c b/net/bluetooth/iso.c
index 4351b0b794e575b9025ed624fca8af580c0a0bc9..6e2923b301505764f9be749173b7aeb2310408ac 100644 (file)
--- a/net/bluetooth/iso.c
+++ b/net/bluetooth/iso.c
@@ -2035,7 +2035,13 @@ static void iso_conn_ready(struct iso_conn *conn)
                }
 
                bacpy(&iso_pi(sk)->dst, &hcon->dst);
-               iso_pi(sk)->dst_type = hcon->dst_type;
+
+               /* Convert from HCI to three-value type */
+               if (hcon->dst_type == ADDR_LE_DEV_PUBLIC)
+                       iso_pi(sk)->dst_type = BDADDR_LE_PUBLIC;
+               else
+                       iso_pi(sk)->dst_type = BDADDR_LE_RANDOM;
+
                iso_pi(sk)->sync_handle = iso_pi(parent)->sync_handle;
                memcpy(iso_pi(sk)->base, iso_pi(parent)->base, iso_pi(parent)->base_len);
                iso_pi(sk)->base_len = iso_pi(parent)->base_len;