Closed Bug 1374146 Opened 3 years ago Closed 3 years ago

Crash in java.util.NoSuchElementException: Death link does not exist at android.os.BinderProxy.unlinkToDeath(Native Method)

Categories

(Firefox for Android :: General, defect)

54 Branch
Unspecified
Android
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Firefox 56
Tracking Status
firefox54 + wontfix
firefox55 --- fixed
firefox56 --- fixed

People

(Reporter: gchang, Assigned: jhlin)

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

This bug was filed from the Socorro interface and is 
report bp-cbdfa27a-5ee1-48ad-a5e9-025960170619.
=============================================================

This is #29 topcrash of fennec in FF54 for android and there is a spike when we enable 99% update rate.

Hi :snorp, 
Can you help find someone for this issue?
Flags: needinfo?(snorp)
Remote codec stuff -> jolin
Flags: needinfo?(snorp) → needinfo?(jolin)
Comment on attachment 8883477 [details]
Bug 1374146 - handle already released death recipient exception.

https://reviewboard.mozilla.org/r/154382/#review159554
Attachment #8883477 - Flags: review?(esawin) → review+
Flags: needinfo?(jolin)
Pushed by jolin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4a39adfb94ae
handle already released death recipient exception. r=esawin
https://hg.mozilla.org/mozilla-central/rev/4a39adfb94ae
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Too late for 54 at this point, but could we please nominate this for Beta approval still?
Assignee: nobody → jolin
Flags: needinfo?(jolin)
Comment on attachment 8883477 [details]
Bug 1374146 - handle already released death recipient exception.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1324530
[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]: none
[Is the change risky?]: low risk
[Why is the change risky/not risky?]: it protects browser from crashing when service disconnection callback executes after remote manager is released, which could happen if remote process dies at the same time when releasing codec.
[String changes made/needed]: none
Flags: needinfo?(jolin)
Attachment #8883477 - Flags: approval-mozilla-beta?
Comment on attachment 8883477 [details]
Bug 1374146 - handle already released death recipient exception.

exception handling to fix a fennec crash, beta55+
Attachment #8883477 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.