Closed Bug 1163945 Opened 9 years ago Closed 9 years ago

Crash [@ _ZThn8_N7mozilla3dom7workers24serviceWorkerScriptCache12_GLOBAL__N_114CompareNetwork14OnStartRequestEP10nsIRequestP11nsISupports ]

Categories

(Core :: DOM: Service Workers, defect)

41 Branch
defect
Not set
critical

Tracking

()

RESOLVED FIXED
mozilla41
Tracking Status
firefox40 + fixed
firefox41 + fixed

People

(Reporter: robin, Assigned: ehsan.akhgari)

References

Details

(Keywords: regression, topcrash)

Crash Data

Attachments

(1 file)

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:41.0) Gecko/20100101 Firefox/41.0
Build ID: 20150511122605

Steps to reproduce:

Not reproducible unfortunately. I left my computer alone for a standup with my usual set of tabs (couple of GMails, Google Calendar, some Github repos etc) and when I came back it had crashed.


Actual results:

Bang: https://crash-stats.mozilla.com/report/index/4d28e709-418c-45ab-8076-8ec7a2150512


Expected results:

No bang.

I wouldn’t normally have bothered filing, but the crash signature looked pretty strange and the 8 other reports ( https://crash-stats.mozilla.com/report/list?product=Firefox&range_unit=days&range_value=3&signature=_ZThn8_N7mozilla3dom7workers24serviceWorkerScriptCache12_GLOBAL__N_114CompareNetwork14OnStartRequestEP10nsIRequestP11nsISupports ) have all happened in the last 3 days so presumably it’s something recent. The signature makes it look like a service worker issue, but feel free to move if it’s not.
Crash Signature: [@ _ZThn8_N7mozilla3dom7workers24serviceWorkerScriptCache12_GLOBAL__N_114CompareNetwork14OnStartRequestEP10nsIRequestP11nsISupports ]
Product: Firefox → Core
Component: Untriaged → DOM: Service Workers
Severity: normal → critical
Possible dupe of #1163404 and #1163593 ?
Bug #1163403 and bug #1163593 even.
Assignee: nobody → ehsan
Blocks: 1164065
I tried to find a way to reproduce this so that I can write a test case for it, so far without success.  But this is clearly happening because we're aborting the CompareNetwork object, and not handling that properly in OnStartRequest.
Marking this as NEW based on comment 5.
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 8605474 [details] [diff] [review]
Handle aborted CompareNetwork jobs properly in CompareNetwork::OnStartRequst

Review of attachment 8605474 [details] [diff] [review]:
-----------------------------------------------------------------

preemptive r+ :)
[Tracking Requested - why for this release]:
This is one of the top 10 crashes we have with people still on builds of 40, so I expect it to be there in Dev Edition as well (unless it's specific to e10s).
Crash Signature: [@ _ZThn8_N7mozilla3dom7workers24serviceWorkerScriptCache12_GLOBAL__N_114CompareNetwork14OnStartRequestEP10nsIRequestP11nsISupports ] → [@ _ZThn8_N7mozilla3dom7workers24serviceWorkerScriptCache12_GLOBAL__N_114CompareNetwork14OnStartRequestEP10nsIRequestP11nsISupports ] [@ mozilla::dom::workers::serviceWorkerScriptCache::`anonymous namespace''::CompareNetwork::OnStartRequest(nsIRequest*, …
It's not specific to e10s, I had the same crash with 41 Nightly and e10s disabled yesterday.
Yeah, this is unrelated to e10s.
Blocks: 1159378
No longer blocks: 1164065
Keywords: regression, topcrash
Comment on attachment 8605474 [details] [diff] [review]
Handle aborted CompareNetwork jobs properly in CompareNetwork::OnStartRequst

(marking as r+ as per comment 8!)
Attachment #8605474 - Flags: review+
Comment on attachment 8605474 [details] [diff] [review]
Handle aborted CompareNetwork jobs properly in CompareNetwork::OnStartRequst

Approval Request Comment
[Feature/regressing bug #]: Bug 1159378
[User impact if declined]: This is a topcrasher on Nightly.
[Describe test coverage new/current, TreeHerder]: None, I couldn't come up with an easy way to test this.
[Risks and why]: Very low risk, just adding a null check.
[String/UUID change made/needed]: none
Attachment #8605474 - Flags: approval-mozilla-aurora?
Since this is a crash, and a regression, enabled tracking for 40 and 41.
Comment on attachment 8605474 [details] [diff] [review]
Handle aborted CompareNetwork jobs properly in CompareNetwork::OnStartRequst

Safe fix. Let's put this directly on Aurora to address the crash before we re-enable updates.
Attachment #8605474 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
https://hg.mozilla.org/mozilla-central/rev/3913a94d7f25
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
(In reply to Lawrence Mandel [:lmandel] (use needinfo) from comment #16)
> Comment on attachment 8605474 [details] [diff] [review]
> Handle aborted CompareNetwork jobs properly in CompareNetwork::OnStartRequst
> 
> Safe fix. Let's put this directly on Aurora to address the crash before we
> re-enable updates.

Awesome, just ran into this on Aurora 41. :)
Jeff, did you run into this crash in Aurora FF41? Can you please re-open the bug if that is the case?
Flags: needinfo?(jgriffiths)
(In reply to Ritu Kothari (:ritu) from comment #21)
> Jeff, did you run into this crash in Aurora FF41? Can you please re-open the
> bug if that is the case?

Not so far, but I will re-open if I see it.
Flags: needinfo?(jgriffiths)
You need to log in before you can comment on or make changes to this bug.