Closed
Bug 1328115
Opened 8 years ago
Closed 8 years ago
Crash in mozilla::jni::Context<T>::ClassRef
Categories
(Core Graveyard :: Widget: Android, defect)
Tracking
(firefox51 unaffected, firefox52 unaffected, firefox53 fixed)
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox51 | --- | unaffected |
firefox52 | --- | unaffected |
firefox53 | --- | fixed |
People
(Reporter: n.nethercote, Assigned: jchen)
References
Details
(Keywords: crash)
Crash Data
Attachments
(3 files)
5.63 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
6.48 KB,
patch
|
jchen
:
review+
|
Details | Diff | Splinter Review |
2.02 KB,
patch
|
snorp
:
review+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is
report bp-68a4f982-d7bb-4554-b5ec-12af42161231.
=============================================================
#1 topcrash in Nightly 20161230030205, though it's only 6 crashes across two installations.
The crash address is always 0xdeadd00d, which is interesting. It's also a startup crash.
jchen, any ideas?
Flags: needinfo?(nchen)
Assignee | ||
Comment 1•8 years ago
|
||
Crashes are happening on API 15, because VsyncSource references several symbols that only exist on API 16+.
Flags: needinfo?(nchen)
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → nchen
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•8 years ago
|
||
Don't go through the VsyncSource Java class to test support for vsync.
Because VsyncSource references symbols from API 16+, Dalvik fails to
load it for API 15 and below. Also make several other changes in
VsyncSource, including removing the unnecessary JNIObject superclass,
and refactoring the instance initialization code, which fixes a possible
race when initializing.
Attachment #8823778 -
Flags: review?(snorp)
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8823779 -
Flags: review+
Assignee | ||
Comment 4•8 years ago
|
||
Be more careful when we try to find Java classes, so that any exceptions
are not ignored and passed on to the caller, which is likely to result
in a hard-to-diagnose Dalvik/ART crash.
Attachment #8823780 -
Flags: review?(snorp)
Attachment #8823778 -
Flags: review?(snorp) → review+
Attachment #8823780 -
Flags: review?(snorp) → review+
Pushed by nchen@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/10968c7a6abf
1. Test vsync support in native code; r=snorp
https://hg.mozilla.org/integration/mozilla-inbound/rev/4ddd36462bdf
2. Update auto-generated bindings; r=me
https://hg.mozilla.org/integration/mozilla-inbound/rev/8f6416404aea
3. Check for exceptions when finding Java class; r=snorp
Comment 6•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/10968c7a6abf
https://hg.mozilla.org/mozilla-central/rev/4ddd36462bdf
https://hg.mozilla.org/mozilla-central/rev/8f6416404aea
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
Updated•8 years ago
|
status-firefox51:
--- → unaffected
status-firefox52:
--- → unaffected
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•