Merge pull request #4662 from Nodraak/fix/4661

[BLE/HRM] Fix #4661

features/FEATURE_BLE/ble/GattCharacteristic.h

@@ -321,6 +321,11 @@ public:
      *        for the value attribute of a characteristic, then that particular
      *        characteristic may be considered optional and dropped while
      *        instantiating the service with the underlying BLE stack.
+     *
+     * @note  A CCCD should not be allocated if either the notify or indicate
+     *        flag is set, as it is handled by the underlying BLE stack. In such
+     *        a case, the param descriptors could be empty and the param
+     *        numDescriptors equal to zero.
      */
     GattCharacteristic(const UUID    &uuid,
                        uint8_t       *valuePtr       = NULL,
@@ -584,6 +589,8 @@ private:
     SecurityManager::SecurityMode_t   _requiredSecurity;
     /**
      * The characteristic's descriptor attributes.
+     * This contains only CCCDs that has neither the notify nor the indicate
+     * flag set, as thoses are handled by the underlying BLE stack.
      */
     GattAttribute                   **_descriptors;
     /**

features/FEATURE_BLE/ble/services/HeartRateService.h

@@ -58,7 +58,7 @@ public:
         valueBytes(hrmCounter),
         hrmRate(GattCharacteristic::UUID_HEART_RATE_MEASUREMENT_CHAR, valueBytes.getPointer(),
                 valueBytes.getNumValueBytes(), HeartRateValueBytes::MAX_VALUE_BYTES,
-                GattCharacteristic::BLE_GATT_CHAR_PROPERTIES_READ | GattCharacteristic::BLE_GATT_CHAR_PROPERTIES_NOTIFY),
+                GattCharacteristic::BLE_GATT_CHAR_PROPERTIES_NOTIFY),
         hrmLocation(GattCharacteristic::UUID_BODY_SENSOR_LOCATION_CHAR, &location),
         controlPoint(GattCharacteristic::UUID_HEART_RATE_CONTROL_POINT_CHAR, &controlPointValue) {
         setupService();