Kasper Lund

_Removed this from the [Later](../milestones/Later) milestone._ _Added [Oldschool-Milestone-Later](../labels/Oldschool-Milestone-Later) label._

_Removed [Oldschool-Milestone-Later](../labels/Oldschool-Milestone-Later) label._

_Removed this from the [Later](../milestones/Later) milestone._ _Added [Oldschool-Milestone-Later](../labels/Oldschool-Milestone-Later) label._

_Removed [Oldschool-Milestone-Later](../labels/Oldschool-Milestone-Later) label._

I found a double free in the BLE implementation and what looks like a few memory leaks. I've attempted to address them here: https://github.com/toitlang/toit/pull/1662.

This looks like it is likely caused by https://github.com/apache/mynewt-nimble/issues/556 because we do not define all services in one go before calling `ble_gatts_start`.