Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SpeechRecognizer mContext leak #87

Closed
Kiwifed0r opened this issue May 14, 2015 · 3 comments · Fixed by #89
Closed

SpeechRecognizer mContext leak #87

Kiwifed0r opened this issue May 14, 2015 · 3 comments · Fixed by #89

Comments

@Kiwifed0r
Copy link 

* (...)Activity has leaked:
* GC ROOT android.speech.SpeechRecognizer$InternalListener.this$0
* references android.speech.SpeechRecognizer.mContext
* leaks (...)Activity instance
* Reference Key: 5204c3b2-9859-4592-ad66-7b3952958f47
* Device: asus google Nexus 7 razor
* Android Version: 4.4.4 API: 19
* Durations: watch=5145ms, gc=318ms, heap dump=965ms, analysis=39984ms
@pyricau
Copy link
Member

pyricau commented May 14, 2015

Steps:

  1. See if it happens in the latest Android (22)
  2. Read the AOSP source, and try to figure out how / why it happens.
  3. Create a repro case
  4. file a bug on b.android.co
  5. add it to AndroidExcludedRefs. Optional: document a hack to fix it

@pyricau
Copy link
Member

pyricau commented May 14, 2015

This was fixed in AOSP here: aosp-mirror/platform_frameworks_base@b37866d

It's fixed in Android 5 only.

@pyricau
Copy link
Member

pyricau commented May 14, 2015

So we only need 5).

pyricau added a commit that referenced this issue May 14, 2015
@pyricau
Ignore SpeechRecognizer leak
9209d8f 
Fixed in AOSP.

Fixes #87
@pyricau pyricau mentioned this issue May 14, 2015
Merged
Yky pushed a commit to Yky/leakcanary that referenced this issue Feb 21, 2016
@pyricau
Ignore SpeechRecognizer leak
49aeb73 
Fixed in AOSP.

Fixes square#87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Ignore SpeechRecognizer leak square/leakcanary
2 participants
@pyricau @Kiwifed0r