#6741 closed defect (fixed)
ANDROID: AssetFdStream deletes ref too early
| Reported by: | SF/manul12 | Owned by: | SF/mthreepwood |
|---|---|---|---|
| Priority: | normal | Component: | Port: Android |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
ScummVM version: 1.7.0 for Android Platform: Android
Bug details: When starting ScummVM on Nexus 7 (2013) on a Android 5.0 "Lollipop" Preview, it first displays a progress bar then exits to homescreen. After that it just always exits to homescreen.
I'm not an Android programmer, but on #ScummVM on IRC I was told to use USB debugging and adb logcat, and it produced the logs you can read in attachment.
Ticket imported from: bugs/6741.
Attachments (1)
Change History (12)
by , 10 years ago
| Attachment: | logcat.txt added |
|---|
comment:1 by , 10 years ago
comment:2 by , 10 years ago
| Owner: | set to |
|---|---|
| Summary: | Android 5.0 Preview - Nexus 7 2013 - crashes to homescreen → ANDROID: AssetFdStream deletes ref too early |
comment:3 by , 10 years ago
fuzzie: Is there an "simple" fix for now, or would this need a major fixup?
comment:4 by , 10 years ago
I don't think that there's any point in making an "easy" fix for this, since the relevant code will be replaced altogether
comment:5 by , 10 years ago
| Owner: | changed from to |
|---|---|
| Resolution: | → fixed |
comment:6 by , 10 years ago
This should now be fixed in b577da2. I don't have a 5.0 device to test, but it was a pretty obvious bug in the code. Let me know with the new daily build (after that commit) if it's still not working for you.
comment:7 by , 10 years ago
| Status: | new → closed |
|---|
comment:8 by , 10 years ago
Those on the forum who have tested it confirmed the bug is fixed. Closing.
comment:9 by , 10 years ago
Good evening. Sorry for the inconvinience but when you would expect this land to Google Play?
comment:10 by , 10 years ago
I'm still seeing this crash on the version currently available on Google Play on my Nexus 6.
comment:11 by , 6 years ago
| Component: | → Port: Android |
|---|
Looks like a really stupid mistake by me (the local ref is getting deleted too early). We should merge pull request #433 which replaces all this code.