Skip to content

Commit ad5bcfc

Browse files
authored
Merge pull request #9427 from paul-szczepanek-arm/fix-truncation
BLE: fix possible truncation
2 parents a23a850 + 001919e commit ad5bcfc

File tree

1 file changed

+4
-3
lines changed
  • features/FEATURE_BLE/targets/TARGET_NORDIC/TARGET_NORDIC_SOFTDEVICE/TARGET_NRF52/source/btle

1 file changed

+4
-3
lines changed

features/FEATURE_BLE/targets/TARGET_NORDIC/TARGET_NORDIC_SOFTDEVICE/TARGET_NRF52/source/btle/btle.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -355,9 +355,10 @@ void btle_handler(const ble_evt_t *p_ble_evt)
355355
uint8_t const data_length_peer =
356356
p_gap_evt->params.data_length_update_request.peer_params.max_tx_octets;
357357

358-
const uint8_t max_data_length = NRF_SDH_BLE_GATT_MAX_MTU_SIZE + 4 /* L2CAP header size */;
359-
360-
uint8_t const data_length = MIN(max_data_length, data_length_peer);
358+
uint8_t const data_length = MIN(
359+
NRF_SDH_BLE_GATT_MAX_MTU_SIZE + 4 /* L2CAP header size */,
360+
data_length_peer
361+
);
361362

362363
ble_gap_data_length_params_t const dlp =
363364
{

0 commit comments

Comments
 (0)