crash in java.lang.NullPointerException: Null native pointer at org.mozilla.gecko.mozglue.GeckoLoader.nativeRun(Native Method)

RESOLVED FIXED in Firefox 46

Status

()

Firefox for Android
General
--
critical
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: WG9s, Assigned: jchen)

Tracking

({crash})

Trunk
Firefox 46
Unspecified
Android
crash
Points:
---

Firefox Tracking Flags

(firefox46 fixed)

Details

(crash signature)

Attachments

(3 attachments)

(Reporter)

Description

2 years ago
This bug was filed from the Socorro interface and is 
report bp-a4640ba0-0617-42fc-a2e5-6bf542151227.
=============================================================
(Assignee)

Updated

2 years ago
Assignee: nobody → nchen
Status: NEW → ASSIGNED
(Assignee)

Comment 1

2 years ago
Created attachment 8703023 [details] [diff] [review]
Always execute a queued call if it's overdue (v1)

GeckoThread.queueNativeCallLocked tries to execute a queued native call
if it's already overdue, but it incorrectly checked that the queue is
not empty. That check is not necessary, and an overdue native call
should always be executed. One-line patch, r=me.
Attachment #8703023 - Flags: review+
(Assignee)

Comment 2

2 years ago
Created attachment 8703024 [details] [diff] [review]
Only prioritize compositor events in GLController (v1)

Only three GLController events -- CreateCompositor, PauseCompositor, and
OnResumedCompositor should be prioritized through GLControllerEvent. The
other GLController events should follow the normal event queue ordering
to prevent race conditions.
Attachment #8703024 - Flags: review?(snorp)
(Assignee)

Comment 3

2 years ago
Created attachment 8703025 [details] [diff] [review]
Configure GeckoLayerClient after GLController is initialized (v1)

LayerView used to call GeckoLayerClient.onGeckoReady directly if Gecko
is sufficiently loaded. However, onGeckoReady indirectly calls
GLController.createCompositor, and it's possible for the
createCompositor event to be prioritized so that it happens before we
initialize GLController, causing a crash. This patch moves the
onGeckoReady call to the Gecko thread, after GLController is
initialized, to avoid this race condition.
Attachment #8703025 - Flags: review?(snorp)
Attachment #8703024 - Flags: review?(snorp) → review+
Attachment #8703025 - Flags: review?(snorp) → review+
(Reporter)

Comment 4

2 years ago
I am not sure why I always find things like this over Holidays.  Sorry for that. I know it takes longer to fix them. But then, but then I do have this antiquated old Samsung Galaxy Tab 2 that I am using that is slower than anything else so seems to find these kind of odd race conditions.
(Assignee)

Comment 5

2 years ago
(In reply to Bill Gianopoulos [:WG9s] from comment #4)
> I am not sure why I always find things like this over Holidays.  Sorry for
> that. I know it takes longer to fix them. But then, but then I do have this
> antiquated old Samsung Galaxy Tab 2 that I am using that is slower than
> anything else so seems to find these kind of odd race conditions.

No, it's good that you found it! I'm sure a lot of users are encountering it based on the crash volume.

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/014d0b989ace
https://hg.mozilla.org/mozilla-central/rev/bd5bcf200648
https://hg.mozilla.org/mozilla-central/rev/6691a39fbeb8
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox46: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 46
Jim - This crash is #8 on Fx44b8. Should we be uplifting this?
Flags: needinfo?(nchen)
(Assignee)

Comment 9

2 years ago
Not this particular bug but I think bug 1236643 will help (different bug but same crash signature)
Flags: needinfo?(nchen)
You need to log in before you can comment on or make changes to this bug.