Closed
Bug 1340160
Opened 8 years ago
Closed 8 years ago
Crash in java.lang.NullPointerException: at org.mozilla.gecko.media.CodecProxy.processInput(CodecProxy.java)
Categories
(Firefox for Android Graveyard :: Audio/Video, defect, P1)
Tracking
(firefox52 unaffected, firefox-esr52 unaffected, firefox53 verified, firefox54 fixed, firefox55 fixed)
RESOLVED
DUPLICATE
of bug 1345599
Firefox 55
Tracking | Status | |
---|---|---|
firefox52 | --- | unaffected |
firefox-esr52 | --- | unaffected |
firefox53 | --- | verified |
firefox54 | --- | fixed |
firefox55 | --- | fixed |
People
(Reporter: marcia, Assigned: jhlin)
References
Details
(Keywords: crash, topcrash)
Crash Data
Attachments
(1 file)
(deleted),
text/x-review-board-request
|
snorp
:
review+
gchang
:
approval-mozilla-aurora+
ritu
:
approval-mozilla-beta+
|
Details |
This bug was filed from the Socorro interface and is
report bp-f51cc1bf-ee28-44ea-b748-6b0962170215.
=============================================================
New crash which started with Build 20170215110205: http://bit.ly/2lSy3KA. Need help bucketing this in the right place so this the ni on snorp.
Possible regression range: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=195049fabb7ac5709e5f75614ba630ba3d1b5a9b&tochange=0a7831d838f793a263456df62f95a615472a7f95
Two Huawei devices seem to be more affected - the Honor 8 and the Nexus 6P.
Flags: needinfo?(snorp)
Comment 1•8 years ago
|
||
#1 topcrash in Nightly 20170217110224, with 26 occurrences it's far ahead of #2, which has 6.
Keywords: topcrash
John, looks like a problem with OOP decoding.
Flags: needinfo?(snorp) → needinfo?(jolin)
Assignee | ||
Comment 3•8 years ago
|
||
Similar to bug 1340582, though different exceptions not caught by the fix there were thrown. Will add more handling code to address this.
Assignee: nobody → jolin
Flags: needinfo?(jolin)
Comment 5•8 years ago
|
||
Still #1 topcrash by a long way in Nightly 20170303110215.
John, do you have an ETA on this? Thank you.
Assignee | ||
Comment 6•8 years ago
|
||
Sorry for the late reply. Will send a patch for review before end of the day.
Flags: needinfo?(jolin)
Comment hidden (mozreview-request) |
Assignee | ||
Comment 8•8 years ago
|
||
Attachment 8843843 [details] will need rebasing if bug 1344347 lands first.
Updated•8 years ago
|
Component: General → Audio/Video
Priority: -- → P1
Comment 9•8 years ago
|
||
mozreview-review |
Comment on attachment 8843843 [details]
Bug 1340160 - catch more local and remote Java exceptions.
https://reviewboard.mozilla.org/r/117448/#review119228
Attachment #8843843 -
Flags: review?(snorp) → review+
Comment 10•8 years ago
|
||
Pushed by jolin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d97a9fd45201
catch more local and remote Java exceptions. r=snorp
Comment 11•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Comment 12•8 years ago
|
||
Please request Aurora/Beta approval on this when you get a chance.
status-firefox52:
--- → unaffected
status-firefox53:
--- → affected
status-firefox-esr52:
--- → unaffected
Flags: needinfo?(jolin)
Comment 13•8 years ago
|
||
No longer crashes (thanks for fixing this), but still not entirely satisfactory - filed bug 1345599 for this.
Assignee | ||
Comment 14•8 years ago
|
||
Comment on attachment 8843843 [details]
Bug 1340160 - catch more local and remote Java exceptions.
Approval Request Comment
[Feature/Bug causing the regression]: bug 1326026
[User impact if declined]: browser crash
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: yes
[Needs manual test from QE? If yes, steps to reproduce]: no
[List of other uplifts needed for the feature/fix]: no
[Is the change risky?]: low risk
[Why is the change risky/not risky?]: it catches more exceptions
[String changes made/needed]: none
Flags: needinfo?(jolin)
Attachment #8843843 -
Flags: approval-mozilla-aurora?
Comment 15•8 years ago
|
||
Comment on attachment 8843843 [details]
Bug 1340160 - catch more local and remote Java exceptions.
Fix a crash. Aurora54+.
Attachment #8843843 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Comment 16•8 years ago
|
||
bugherder uplift |
Updated•8 years ago
|
Attachment #8843843 -
Flags: approval-mozilla-beta?
Comment on attachment 8843843 [details]
Bug 1340160 - catch more local and remote Java exceptions.
Browser crash fix, Beta53+
Attachment #8843843 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 18•8 years ago
|
||
bugherder uplift |
Comment 19•8 years ago
|
||
Saw this crash again on Aurora 54.0a2 (2017-03-09), while waiting for this page to load: https://direlogomachist.itch.io/ember-interlude. Occurred 4/4 times.
Crash report: https://crash-stats.mozilla.com/report/index/45a9d016-0ae2-4aad-85b9-8cc812170310
Devices:
Nexus 6 (Android 7.0)
Asus ZenPad 8.0 Z380KL (Android 6.0.1)
note: Also crashed on Nightly 55, but could only reproduce it once.
On Beta 53, it did not reproduce.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Updated•8 years ago
|
Comment 20•8 years ago
|
||
Tested with Nexus 6 (Android 7.0) and Huawei MediaPad M2 (Android 5.1.1) on Firefox 53 Beta 2 and didn't saw this crash. Mark the flag verified.
Updated•8 years ago
|
Flags: needinfo?(jolin)
Assignee | ||
Comment 21•8 years ago
|
||
Some log shows shared memory error and looks like what bug 1345599 tries to fix. Set it as a dup.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 years ago
Flags: needinfo?(jolin)
Resolution: --- → DUPLICATE
Updated•8 years ago
|
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•