Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Factory and Provider Memory Leak

See original GitHub issue

I added LeakCanary to my app, and I’ve been getting reports that any class that uses a factory or provider binding is leaking a reference to a destroyed Activity.

I didn’t look too deeply into it, but it looks like the lambda in Factory.getFactory is capturing the BindingDI instance, which for some reason has a reference to my Activity in its context field. I’m not sure how the activity got set as the context.

I use this scope but nothing in SomethingApi, SomethingManager, nor somethingFactory use it.

There is a class bound in an activitySingleton scope that uses SomethingManager but that’s not referenced in the leak trace, it is bound after SomethingApi, and I’ve verified that it and activityScope has been properly destroyed.

┬───
│ GC Root: System class
│
├─ android.provider.FontsContract class
│    Leaking: NO (MyApp↓ is not leaking and a class is never leaking)
│    ↓ static FontsContract.sContext
├─ co.my.app.MyApp instance
│    Leaking: NO (Application is a singleton)
│    MyApp does not wrap an activity context
│    ↓ MyApp.kodein
│                     ~~~~~~
├─ org.kodein.di.LazyKodein instance
│    Leaking: UNKNOWN
│    ↓ LazyKodein.baseKodein$delegate
│                 ~~~~~~~~~~~~~~~~~~~
├─ kotlin.SynchronizedLazyImpl instance
│    Leaking: UNKNOWN
│    ↓ SynchronizedLazyImpl._value
│                           ~~~~~~
├─ org.kodein.di.internal.KodeinImpl instance
│    Leaking: UNKNOWN
│    ↓ KodeinImpl._container
│                 ~~~~~~~~~~
├─ org.kodein.di.internal.KodeinContainerImpl instance
│    Leaking: UNKNOWN
│    ↓ KodeinContainerImpl.tree
│                          ~~~~
├─ org.kodein.di.internal.KodeinTreeImpl instance
│    Leaking: UNKNOWN
│    ↓ KodeinTreeImpl.bindings
│                     ~~~~~~~~
├─ java.util.HashMap instance
│    Leaking: UNKNOWN
│    ↓ HashMap.table
│              ~~~~~
├─ java.util.HashMap$Node[] array
│    Leaking: UNKNOWN
│    ↓ HashMap$Node[].[89]
│                     ~~~~
├─ java.util.HashMap$Node instance
│    Leaking: UNKNOWN
│    ↓ HashMap$Node.value
│                   ~~~~~
├─ java.util.ArrayList instance
│    Leaking: UNKNOWN
│    ↓ ArrayList.elementData
│                ~~~~~~~~~~~
├─ java.lang.Object[] array
│    Leaking: UNKNOWN
│    ↓ Object[].[0]
│               ~~~
├─ org.kodein.di.KodeinDefinition instance
│    Leaking: UNKNOWN
│    ↓ KodeinDefinition.binding
│                       ~~~~~~~
├─ org.kodein.di.bindings.Singleton instance
│    Leaking: UNKNOWN
│    ↓ Singleton.scope
│                ~~~~~
├─ org.kodein.di.bindings.NoScope instance
│    Leaking: UNKNOWN
│    ↓ NoScope._registry
│              ~~~~~~~~~
├─ org.kodein.di.bindings.StandardScopeRegistry instance
│    Leaking: UNKNOWN
│    ↓ StandardScopeRegistry._cache
│                            ~~~~~~
├─ java.util.concurrent.ConcurrentHashMap instance
│    Leaking: UNKNOWN
│    ↓ ConcurrentHashMap.table
│                        ~~~~~
├─ java.util.concurrent.ConcurrentHashMap$Node[] array
│    Leaking: UNKNOWN
│    ↓ ConcurrentHashMap$Node[].[0]
│                               ~~~
├─ java.util.concurrent.ConcurrentHashMap$Node instance
│    Leaking: UNKNOWN
│    ↓ ConcurrentHashMap$Node.val
│                             ~~~
├─ org.kodein.di.bindings.SingletonReference$make$1 instance
│    Leaking: UNKNOWN
│    Anonymous subclass of kotlin.jvm.internal.Lambda
│    ↓ SingletonReference$make$1.$value
│                                ~~~~~~
├─ co.my.app.api.SomethingApiImpl instance
│    Leaking: UNKNOWN
│    ↓ SomethingApiImpl.somethingManager
│                    ~~~~~~~~~~~~~
├─ co.my.app.models.something.SomethingManagerImpl instance
│    Leaking: UNKNOWN
│    ↓ SomethingManagerImpl.somethingFactory
│                        ~~~~~~~~~~~~~
├─ org.kodein.di.bindings.Factory$getFactory$1 instance
│    Leaking: UNKNOWN
│    Anonymous subclass of kotlin.jvm.internal.Lambda
│    ↓ Factory$getFactory$1.$kodein
│                           ~~~~~~~
├─ org.kodein.di.internal.BindingKodeinImpl instance
│    Leaking: UNKNOWN
│    ↓ BindingKodeinImpl.context
│                        ~~~~~~~
╰→ co.my.app.main.MainActivity instance
     Leaking: YES (ObjectWatcher was watching this because co.myapp.main.MainActivity received Activity#onDestroy() callback and Activity#mDestroyed is true)