From 31e028f22ff31ccc56edea671e708d257e39130b Mon Sep 17 00:00:00 2001 From: Vincent Coubard Date: Sun, 11 Nov 2018 18:03:06 +0000 Subject: [PATCH] BLE: fix integer overflow in setPeriodicAdvertisingParameters --- features/FEATURE_BLE/source/generic/GenericGap.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/features/FEATURE_BLE/source/generic/GenericGap.cpp b/features/FEATURE_BLE/source/generic/GenericGap.cpp index ed1256449a..65c4481893 100644 --- a/features/FEATURE_BLE/source/generic/GenericGap.cpp +++ b/features/FEATURE_BLE/source/generic/GenericGap.cpp @@ -1932,8 +1932,8 @@ ble_error_t GenericGap::setPeriodicAdvertisingParameters( bool advertiseTxPower ) { - uint16_t interval_min = (periodicAdvertisingIntervalMinMs * 100) / 125; - uint16_t interval_max = (periodicAdvertisingIntervalMinMs * 100) / 125; + uint32_t interval_min = (periodicAdvertisingIntervalMinMs * 100) / 125; + uint32_t interval_max = (periodicAdvertisingIntervalMaxMs * 100) / 125; if (interval_min < 6 || interval_min > 0xFFFF) { return BLE_ERROR_INVALID_PARAM;