espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-10-02 21:27:22 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(nimble):Handled the Load access fault crash caused due to an invalid setting of index-varible 'reattempt_idx'.

Browse Source
This commit is contained in:
Abhinav Kudnar
2024-02-20 12:22:49 +05:30
parent d0ad889f56
commit 28b42f4f14
1 changed files with 10 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/host/src/ble_gap.c
+10 -2
View File
@@ -134,6 +134,7 @@ static uint16_t reattempt_idx;
static bool conn_cookie_enabled;
#endif
/**
* The state of the in-progress master connection. If no master connection is
* currently in progress, then the op field is set to BLE_GAP_OP_NULL.
@@ -5737,8 +5738,15 @@ ble_gap_connect(uint8_t own_addr_type, const ble_addr_t *peer_addr,
/* ble_gap_connect_reattempt save the connection parameters */
if ((cb_arg != NULL) && conn_cookie_enabled) {
struct ble_gap_conn_desc *conn_desc = cb_arg;
/* reattempt_idx should follow conn handle corresponding to MASTER role */
reattempt_idx = conn_desc->conn_handle;
struct ble_gap_conn_desc *curr_conn_desc=NULL;
/* reattempt_idx is set to that index where corresponding conn_handle entry was made */
for (int i = 0; i < MYNEWT_VAL(BLE_MAX_CONNECTIONS); i++) {
curr_conn_desc = ble_conn_reattempt[i].cb_arg;
if (curr_conn_desc && (conn_desc->conn_handle == curr_conn_desc->conn_handle)) {
reattempt_idx = i;
break;
}
}
/* Reset cookie_enabled flag, it will be set again by reattempt call */
conn_cookie_enabled = false;
}