Leak of Activity by ApiDispatcher
See original GitHub issueWhen running with LeakCanary, it’s detecting a leak
* GC ROOT static com.microsoft.identity.client.internal.controllers.MSALApiDispatcher.sCommand
* references com.microsoft.identity.client.internal.controllers.MSALInteractiveTokenCommand.mParameters
* references com.microsoft.identity.client.internal.controllers.MSALAcquireTokenOperationParameters.mActivity
A potential fix would be to clear out sCommand after completeInteractive
is called
public static void completeInteractive(int requestCode, int resultCode, final Intent data) {
sCommand.notify(requestCode, resultCode, data);
+ sCommand = null;
}
Issue Analytics
- State:
- Created 5 years ago
- Reactions:4
- Comments:6 (4 by maintainers)
Top Results From Across the Web
Why LeakCanary reports about leak of each activity's field ...
Why LeakCanary reports about leak of each activity's field even right after activity was just started? Ask Question. Asked 5 years, 11 months ......
Read more >Potential library leak in Android SDK's transition #2087 - GitHub
ActivityThread class │ Leaking: NO (OnboardingActivity↓ is not leaking ... NO (SplashFragment↓ is not leaking and Activity#mDestroyed is ...
Read more >How to get rid of memory leaks? A practical approach using ...
1 . Detecting retained objects. LeakCanary hooks into the Android lifecycle and detects when activities and fragments are destroyed and should be garbage ......
Read more >Code recipes - LeakCanary
Watching objects with a lifecycle¶. The default configuration of LeakCanary will automatically watch Activity, Fragment, Fragment View and ViewModel instances.
Read more >Simple Index
a-pandas-ex-adb-execute-activities · a-pandas-ex-adb-settings-to-df · a-pandas-ex-adb-to-df ... api-dispatcher · api-display-purposes ... auto-py-torrent
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
PR with proposed fix open here: https://github.com/AzureAD/microsoft-authentication-library-common-for-android/pull/580
Merged. This will ship in an upcoming release.