FlutterBleLib Leaking BleAdapter

If you create multiple clients when FlutterBleLibPlugin.createClient() overwrites the existing BleAdapter.

This means you can never destroy the previous clients, I'm pretty sure that this could cause a memory leaks as there are listeners attached etc.

If you guard against creating a second one then bleAdapter.createClient() will need to be skipped also as it will cause the same issue with leaks as BleModule is written assuming the same pattern which will overwrite the client, see BleModule.java#L110

Dec 16 '20 22:12 JamesMcIntosh

@JamesMcIntosh did you test if this happens with the memory profiler?

Jan 22 '21 18:01 duck-dev-go

Hi @michael-ottink, I don't think that I checked it with a profiler, I came to this suggestion from a visual inspection of the code and the reported behaviour around hot restarts.

Jan 24 '21 21:01 JamesMcIntosh

I profiled it and it is leaking as expected.

Mar 20 '21 11:03 JamesMcIntosh

To get around the hot restart issue the createClient method can either destroy and recreate or reuse the existing one.

After testing out the destroyClient method and it looks like the destroy it still leaks.

The leak originates in MultiPlatformBleAdapter is because of RxJava1's subscriptions to Observers holds strong references to the anonymous inner classes inside BleModule i.e onError and onNext.

More information: https://medium.com/@scanarch/how-to-leak-memory-with-subscriptions-in-rxjava-ae0ef01ad361

Mar 20 '21 12:03 JamesMcIntosh