Add checks in case NIMBLE_CPP_DEBUG_ASSERT is not defined.

thekurtovic · thekurtovic · commit a8da04794e32 · 2025-01-03T13:56:27.000-05:00
diff --git a/src/NimBLEAttValue.cpp b/src/NimBLEAttValue.cpp
@@ -26,6 +26,8 @@
 
 # include "NimBLEAttValue.h"
 
+static const char* LOG_TAG = "NimBLEAttValue";
+
 // Default constructor implementation.
 NimBLEAttValue::NimBLEAttValue(uint16_t init_len, uint16_t max_len)
     : m_attr_value{static_cast<uint8_t*>(calloc(init_len + 1, 1))},
@@ -38,6 +40,9 @@ NimBLEAttValue::NimBLEAttValue(uint16_t init_len, uint16_t max_len)
 # endif
 {
     NIMBLE_CPP_DEBUG_ASSERT(m_attr_value);
+    if (m_attr_value == nullptr) {
+        NIMBLE_LOGE(LOG_TAG, "Failed to calloc ctx");
+    }
 }
 
 // Value constructor implementation.
@@ -57,6 +62,10 @@ NimBLEAttValue::~NimBLEAttValue() {
 // Move assignment operator implementation.
 NimBLEAttValue& NimBLEAttValue::operator=(NimBLEAttValue&& source) {
     if (this != &source) {
+        if (m_attr_value == nullptr) {
+            NIMBLE_LOGE(LOG_TAG, "Move assign nullptr");
+            return *this;
+        }
         free(m_attr_value);
         m_attr_value   = source.m_attr_value;
         m_attr_max_len = source.m_attr_max_len;
@@ -81,6 +90,10 @@ NimBLEAttValue& NimBLEAttValue::operator=(const NimBLEAttValue& source) {
 void NimBLEAttValue::deepCopy(const NimBLEAttValue& source) {
     uint8_t* res = static_cast<uint8_t*>(realloc(m_attr_value, source.m_capacity + 1));
     NIMBLE_CPP_DEBUG_ASSERT(res);
+    if (res == nullptr) {
+        NIMBLE_LOGE(LOG_TAG, "Failed to realloc deepCopy");
+        return;
+    }
 
     ble_npl_hw_enter_critical();
     m_attr_value   = res;
@@ -106,7 +119,7 @@ NimBLEAttValue& NimBLEAttValue::append(const uint8_t* value, uint16_t len) {
     }
 
     if ((m_attr_len + len) > m_attr_max_len) {
-        NIMBLE_LOGE("NimBLEAttValue", "val > max, len=%u, max=%u", len, m_attr_max_len);
+        NIMBLE_LOGE(LOG_TAG, "val > max, len=%u, max=%u", len, m_attr_max_len);
         return *this;
     }
 
@@ -117,6 +130,10 @@ NimBLEAttValue& NimBLEAttValue::append(const uint8_t* value, uint16_t len) {
         m_capacity = new_len;
     }
     NIMBLE_CPP_DEBUG_ASSERT(res);
+    if (res == nullptr) {
+        NIMBLE_LOGE(LOG_TAG, "Failed to realloc append");
+        return *this;
+    }
 
 # if CONFIG_NIMBLE_CPP_ATT_VALUE_TIMESTAMP_ENABLED
     time_t t = time(nullptr);
diff --git a/src/NimBLEAttValue.h b/src/NimBLEAttValue.h
@@ -325,6 +325,10 @@ class NimBLEAttValue {
     /** @brief Subscript operator */
     uint8_t operator[](int pos) const {
         NIMBLE_CPP_DEBUG_ASSERT(pos < m_attr_len);
+        if (pos >= m_attr_len) {
+            NIMBLE_LOGE("NimBLEAttValue", "pos >= len, pos=%u, len=%u", pos, m_attr_len);
+            return 0;
+        }
         return m_attr_value[pos];
     }
 
diff --git a/src/NimBLEDevice.cpp b/src/NimBLEDevice.cpp
@@ -1231,7 +1231,7 @@ std::string NimBLEDevice::toString() {
  * @param [in] line The line number where the assert occurred.
  */
 void nimble_cpp_assert(const char* file, unsigned line) {
-    console_printf("Assertion failed at %s:%u\n", file, line);
+    NIMBLE_LOGE("", "Assertion failed at %s:%u\n", file, line);
     ble_npl_time_delay(10);
     abort();
 }

Original file line number	Diff line number	Diff line change
`@@ -1231,7 +1231,7 @@ std::string NimBLEDevice::toString() {`
`1231`	`1231`	`* @param [in] line The line number where the assert occurred.`
`1232`	`1232`	`*/`
`1233`	`1233`	`void nimble_cpp_assert(const char* file, unsigned line) {`
`1234`		`- console_printf("Assertion failed at %s:%u\n", file, line);`
	`1234`	`+ NIMBLE_LOGE("", "Assertion failed at %s:%u\n", file, line);`
`1235`	`1235`	`ble_npl_time_delay(10);`
`1236`	`1236`	`abort();`
`1237`	`1237`	`}`