NimBLEDevice::deinit() Memory Leak

[justcallmekoko]

There appears to be a memory leak when deinitializing NimBLEDevice.

The following code demonstrates the memory leak. This is using ESP32-C5 and NimBLE commit b425a1a

Edit: You should see available memory slowly going down until eventual crash.

Edit 2: Passing true to NimBLEDevice::deinit() does not resolve the issue.

#include <NimBLEDevice.h>

void setup() {
  Serial.begin(115200);

  while(!Serial) delay(100);
}

void loop() {
  Serial.println(ESP.getFreeHeap());
  NimBLEDevice::init("");
  delay(100);
  NimBLEDevice::deinit();
}

[mLihs]

I had the same issue. fix is:

Since version 3.3.7, the ESP32 core uses btInUse() to check if Bluetooth is being used before startup.
Only libraries that include esp32-hal-bt-mem.h will report themselves as Bluetooth users.
The core's internal BLE library does this, but the external library NimBLE-Arduino does not.
As a result, the core's BLE remains unused, freeing up the Bluetooth controller memory.
When NimBLEDevice::init() is called later, this memory is missing, and the heap is modified → TLSF assertion.

#if defined(ARDUINO_ARCH_ESP32)
#include "esp32-hal-bt-mem.h"
#endif

or

extern "C" bool btInUse(void) {
return true;
}

[justcallmekoko]

@mLihs

I have tried both suggestions but the available heap still seems to go down with each new init/deinit

[h2zero]

Thanks @justcallmekoko, I believe this to be an upstream issue, I will look into fixing it when I have a chance.

[justcallmekoko]

Sounds good. Thank you for the awesome work!