]> granicus.if.org Git - esp-idf/commitdiff
prevent heap_caps_add_region exception if region is empty
authorme-no-dev <hristo@espressif.com>
Fri, 17 Aug 2018 06:49:48 +0000 (08:49 +0200)
committerme-no-dev <hristo@espressif.com>
Fri, 17 Aug 2018 06:49:48 +0000 (08:49 +0200)
Calling `esp_bt_controller_mem_release` or `esp_bt_mem_release` might result in exception if any of the regions returned to heap are empty. This happens usually if BT/BLE is enabled, but not used/linked in the final firmware.

components/bt/bt.c

index ba30dd861a83d49b2ef904a57db70be78f097210..a5d5e14249a64d92affe1ed668627a1915469996 100644 (file)
@@ -885,12 +885,16 @@ esp_err_t esp_bt_controller_mem_release(esp_bt_mode_t mode)
     if (mode == ESP_BT_MODE_BTDM) {
         mem_start = (intptr_t)&_btdm_bss_start;
         mem_end = (intptr_t)&_btdm_bss_end;
-        ESP_LOGD(BTDM_LOG_TAG, "Release BTDM BSS [0x%08x] - [0x%08x]\n", mem_start, mem_end);
-        ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        if (mem_start != mem_end) {
+            ESP_LOGD(BTDM_LOG_TAG, "Release BTDM BSS [0x%08x] - [0x%08x]\n", mem_start, mem_end);
+            ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        }
         mem_start = (intptr_t)&_btdm_data_start;
         mem_end = (intptr_t)&_btdm_data_end;
-        ESP_LOGD(BTDM_LOG_TAG, "Release BTDM Data [0x%08x] - [0x%08x]\n", mem_start, mem_end);
-        ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        if (mem_start != mem_end) {
+            ESP_LOGD(BTDM_LOG_TAG, "Release BTDM Data [0x%08x] - [0x%08x]\n", mem_start, mem_end);
+            ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        }
     }
     return ESP_OK;
 }
@@ -908,12 +912,16 @@ esp_err_t esp_bt_mem_release(esp_bt_mode_t mode)
     if (mode == ESP_BT_MODE_BTDM) {
         mem_start = (intptr_t)&_bt_bss_start;
         mem_end = (intptr_t)&_bt_bss_end;
-        ESP_LOGD(BTDM_LOG_TAG, "Release BT BSS [0x%08x] - [0x%08x]\n", mem_start, mem_end);
-        ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        if (mem_start != mem_end) {
+            ESP_LOGD(BTDM_LOG_TAG, "Release BT BSS [0x%08x] - [0x%08x]\n", mem_start, mem_end);
+            ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        }
         mem_start = (intptr_t)&_bt_data_start;
         mem_end = (intptr_t)&_bt_data_end;
-        ESP_LOGD(BTDM_LOG_TAG, "Release BT Data [0x%08x] - [0x%08x]\n", mem_start, mem_end);
-        ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        if (mem_start != mem_end) {
+            ESP_LOGD(BTDM_LOG_TAG, "Release BT Data [0x%08x] - [0x%08x]\n", mem_start, mem_end);
+            ESP_ERROR_CHECK(heap_caps_add_region(mem_start, mem_end));
+        }
     }
     return ESP_OK;
 }