Closed Bug 720622 Opened 12 years ago Closed 3 years ago

Avoid crashing when library loading fails somehow

Categories

(Firefox for Android Graveyard :: General, enhancement, P3)

ARM
Android
enhancement

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: glandium, Unassigned)

Details

(Whiteboard: [lang=java])

Currently, when the linker fails to load libraries, Fennec just crashes. Considering we have a native UI, we could avoid that and do something useful instead, like displaying a message, exiting properly, and/or even allow to download the next nightly.
tracking-fennec: --- → ?
tracking-fennec: ? → +
Mike, No one is looking at this.  Is this something you prototyped up?
Severity: normal → enhancement
Priority: -- → P3
No, I just have patches that delay the crash. (that is avoid a crash in loadLibs, but then crash when JNI is called from java code)
Whiteboard: [mentor=kats]
I'm presuming about the language here; feel free to correct me.
Whiteboard: [mentor=kats] → [mentor=kats][language=java]
Yup. And actually I realize now somebody from the frontend team would be better since this mostly involves throwing up some pretty Android UI and shutting down properly.
Whiteboard: [mentor=kats][language=java] → [mentor=bnicholson][language=java]
Whiteboard: [mentor=bnicholson][language=java] → [mentor=bnicholson][lang=java]
So what would we want here? Just some dialog with a string and an OK button?

Via IRC, Kats said that one possible cause of this lib load failing would be loading an ARMv7 build on an ARMv6 device, but that there is no way to distinguish that from other causes of the same crash. So is there anything useful we could actually give to the user at this point to help figure out the problem? If not, I'm not sure I see how this is an improvement over the crash reporter UI.
In that case you wouldn't get the crash reporter UI.
Mentor: bnicholson
Whiteboard: [mentor=bnicholson][lang=java] → [lang=java]
I think this is fixed via bug 990130.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
AIUI, bug 990130 handled the case of the mozglue library failing to load, not gecko libraries.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Mentor: bnicholson
Since the last update is more than 2 years, we are removing the tracking-fennec flag for now.
Feel free to renominate the bug if it happens again and it's reproducible. 

Thank you !
tracking-fennec: + → ---
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: REOPENED → RESOLVED
Closed: 10 years ago3 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.