espressif/esp-nimble
1
0
Fork 0
mirror of https://github.com/espressif/esp-nimble.git synced 2026-09-13 20:50:00 +00:00
Code Issues Packages Projects Releases Wiki Activity

nimble/ll: Fix handling RX data while encryption as a Slave

Browse Source 
Set enc_stat in the LL_ENC_REQ handler and not when sending LL_ENC_RSP.
This is in order to make sure that next incoming PDU will be
alread verified

This fix small rate of failures of LL/SEC/SLA/BI-05
This commit is contained in:
Łukasz Rymanowski
2019-02-07 20:51:14 +01:00
parent 664bf53ae1
commit 44569e1c72
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
nimble/controller/src/ble_ll_ctrl.c
+2 -1
View File
@@ -1295,6 +1295,8 @@ ble_ll_ctrl_rx_enc_req(struct ble_ll_conn_sm *connsm, uint8_t *dptr,
return BLE_LL_CTRL_UNKNOWN_RSP; return BLE_LL_CTRL_UNKNOWN_RSP;
} }
connsm->enc_data.enc_state = CONN_ENC_S_LTK_REQ_WAIT;
/* In case we were already encrypted we need to reset packet counters */ /* In case we were already encrypted we need to reset packet counters */
connsm->enc_data.rx_pkt_cntr = 0; connsm->enc_data.rx_pkt_cntr = 0;
connsm->enc_data.tx_pkt_cntr = 0; connsm->enc_data.tx_pkt_cntr = 0;
@@ -2621,7 +2623,6 @@ ble_ll_ctrl_tx_done(struct os_mbuf *txpdu, struct ble_ll_conn_sm *connsm)
connsm->enc_data.enc_state = CONN_ENC_S_ENC_RSP_WAIT; connsm->enc_data.enc_state = CONN_ENC_S_ENC_RSP_WAIT;
break; break;
case BLE_LL_CTRL_ENC_RSP: case BLE_LL_CTRL_ENC_RSP:
connsm->enc_data.enc_state = CONN_ENC_S_LTK_REQ_WAIT;
connsm->csmflags.cfbit.send_ltk_req = 1; connsm->csmflags.cfbit.send_ltk_req = 1;
break; break;
case BLE_LL_CTRL_START_ENC_RSP: case BLE_LL_CTRL_START_ENC_RSP: