[Scummvm-tracker] [ScummVM :: Bugs] #6387: ANDROID: Error handling ends in segfault

[ScummVM :: Bugs]

#6387: ANDROID: Error handling ends in segfault
----------------------------+---------------------------
  Reporter:  SF/mschomburg  |      Owner:  fuzzie
      Type:  defect         |     Status:  new
  Priority:  normal         |  Component:  Port: Android
Resolution:                 |   Keywords:
      Game:                 |
----------------------------+---------------------------
Comment (by antoniou79):

 I've tested today with the plain (non-SDL) Android port (local build
 ScummVM 2.2.0git (targeting armeabi-v7a only) on three different devices

 - a Huawei, Android 4.4.4
 - a Moto E, Android 5.1
 - and a Xperia Z5 Compact, Android 7.1

 I also tested with today's stable build from the ScummVM buildbot for
 Android (ARM64) (ScummVM 2.1.0pre11-gf4638bbfc3 (Sep 25 2019 06:43:41))
 on:

 - the Xperia Z5 Compact, Android 7.1 device.

 The test case was not with the t7g game but with Beneath a Steel Sky CD
 version (downloadable from ScummVM Games web page), since scott-t
 mentioned that the issue could be reproduced using that game (and in fact
 should be independent of the game engine).

 The BASS game runs ok and exits ok if it has all of its required data
 files on all three devices.
 By removing a required data file (eg sky.dnr) the app will exit to
 "desktop" but without a seg fault, again on all three devices.

 logcat indicates:
 {{{E/ScummVM: Could not open sky.dnr!}}}

 No follow up message about a segmentation fault / SIGSEGV or similar, and
 there's no error popup on the devices.

 There's also no popup informative message on the devices about why ScummVM
 just exited upon loading the game, but that is beyond the scope of this
 bug ticket.

 I am unsure as of yet if this got fixed by my recent commits or earlier
 ones.
 For reference these are my recent commits:
 https://github.com/scummvm/scummvm/commit/d39eca7e39f322b378845aba9ce7c09bc477987d
 https://github.com/scummvm/scummvm/commit/b4b9ad1743dcf97999b7fda4860aa964d119118c
 https://github.com/scummvm/scummvm/commit/f79d5d9b6baa3e002c93df22d29eac62de3e618f

 The commits are backported to 2.1 branch.

 I would prefer if someone also tested on their devices, preferably with
 the v8a 64 bit build, since it has happened before that the android app
 would not crash on any of my devices but it would crash for someone else!
-- 
Ticket URL: <https://bugs.scummvm.org/ticket/6387#comment:21>
ScummVM :: Bugs <https://bugs.scummvm.org>
ScummVM