Menu

#6741 ANDROID: AssetFdStream deletes ref too early

*None
closed-fixed
None
5
2015-03-31
2014-10-26
Manul
No

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.

1 Attachments

Discussion

  • fuzzie

    fuzzie - 2014-10-26

    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.

     
  • fuzzie

    fuzzie - 2014-10-26
    • summary: Android 5.0 Preview - Nexus 7 2013 - crashes to homescreen --> ANDROID: AssetFdStream deletes ref too early
    • assigned_to: Alyssa Milburn
     
  • digitall

    digitall - 2014-10-26

    fuzzie: Is there an "simple" fix for now, or would this need a major fixup?

     
  • Filippos Karapetis

    I don't think that there's any point in making an "easy" fix for this, since the relevant code will be replaced altogether

     
  • clone2727

    clone2727 - 2014-11-15
    • status: open --> open-fixed
    • assigned_to: Alyssa Milburn --> clone2727
     
  • clone2727

    clone2727 - 2014-11-15

    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.

     
  • clone2727

    clone2727 - 2014-11-16
    • status: open-fixed --> closed-fixed
     
  • clone2727

    clone2727 - 2014-11-16

    Those on the forum who have tested it confirmed the bug is fixed. Closing.

     
  • Tuomas Soikkeli

    Tuomas Soikkeli - 2015-03-07

    Good evening. Sorry for the inconvinience but when you would expect this land to Google Play?

     
  • Haradan

    Haradan - 2015-03-31

    I'm still seeing this crash on the version currently available on Google Play on my Nexus 6.