Commit dd7caf66 authored by Aleksander Wasaznik's avatar Aleksander Wasaznik Committed by Fabio Baltieri
Browse files

Bluetooth: Host: Set `conn->err` in prio 



The goal is to not access `buf` in the non-prio `hci_disconn_complete`
so that the buffer does not need to be retained.

This commit moves the copy of `conn->err` to the prio handler.

Signed-off-by: default avatarAleksander Wasaznik <aleksander.wasaznik@nordicsemi.no>
parent f36ddf04

subsys/bluetooth/host/hci_core.c

+2 −2
Original line number Diff line number Diff line
@@ -854,6 +854,8 @@ static void hci_disconn_complete_prio(struct net_buf *buf) 
		return;

	}



	conn->err = evt->reason;



	bt_conn_set_state(conn, BT_CONN_DISCONNECT_COMPLETE);

	bt_conn_unref(conn);

}
@@ -876,8 +878,6 @@ static void hci_disconn_complete(struct net_buf *buf) 
		return;

	}



	conn->err = evt->reason;



	bt_conn_set_state(conn, BT_CONN_DISCONNECTED);



	if (conn->type != BT_CONN_TYPE_LE) {

CRA Git | Maintained and supported by SUSTech CRA and CCSE