Crash in java.lang.IllegalStateException: at android.os.Parcel.readException(Parcel.java)

RESOLVED FIXED in Firefox 53

Status

()

Firefox for Android
Audio/Video
--
critical
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: njn, Assigned: jolin)

Tracking

({crash})

unspecified
Firefox 54
Unspecified
Android
crash
Points:
---

Firefox Tracking Flags

(firefox52 unaffected, firefox53 fixed, firefox54 fixed)

Details

(crash signature)

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

10 months ago
This bug was filed from the Socorro interface and is 
report bp-e57b80e3-329e-4727-a8f9-711b12170212.
=============================================================

This is the #1 topcrash in Aurora 20170210084116, with 25 occurrences. It's also showing up a bit on Nightly. There's a bit of variety in the Java stack traces, though they all seem to go through ICodec.java.

Looks like it first started happening in January.

Jim, any ideas?
Flags: needinfo?(nchen)
Flags: needinfo?(nchen) → needinfo?(jolin)
(Assignee)

Comment 1

10 months ago
Looks like some exceptions were not caught in remote decoder and leaked to local proxy.
Assignee: nobody → jolin
Flags: needinfo?(jolin)
Comment hidden (mozreview-request)

Comment 3

10 months ago
mozreview-review
Comment on attachment 8837404 [details]
Bug 1338975 - Don't leak MediaCodec exceptions to CodecProxy.

https://reviewboard.mozilla.org/r/112544/#review114298
Attachment #8837404 - Flags: review?(nchen) → review+

Comment 4

10 months ago
Pushed by jolin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/959ce2584a80
Don't leak MediaCodec exceptions to CodecProxy. r=jchen

Comment 5

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/959ce2584a80
Status: NEW → RESOLVED
Last Resolved: 10 months ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Please request Aurora approval on this when you get a chance.
status-firefox52: --- → unaffected
status-firefox53: --- → affected
Flags: needinfo?(jolin)
(Assignee)

Comment 7

10 months ago
Comment on attachment 8837404 [details]
Bug 1338975 - Don't leak MediaCodec exceptions to CodecProxy.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1338975
[User impact if declined]: browser crash when playing video/audio.
[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]: none
[Is the change risky?]: low risk
[Why is the change risky/not risky?]: it catches more exceptions from codec 
[String changes made/needed]: none
Flags: needinfo?(jolin)
Attachment #8837404 - Flags: approval-mozilla-aurora?

Comment 8

10 months ago
Comment on attachment 8837404 [details]
Bug 1338975 - Don't leak MediaCodec exceptions to CodecProxy.

Fix a crash. Aurora53+.
Attachment #8837404 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+

Comment 9

10 months ago
bugherderuplift
https://hg.mozilla.org/releases/mozilla-aurora/rev/c4dda37794c1
status-firefox53: affected → fixed
You need to log in before you can comment on or make changes to this bug.