Closed
Bug 1351327
Opened 8 years ago
Closed 6 years ago
Crash in mozalloc_abort | Abort | NS_DebugBreak | mozilla::ipc::FatalError | mozilla::dom::PContentChild::Read
Categories
(Core :: DOM: Service Workers, defect, P3)
Tracking
()
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox53 | --- | affected |
firefox54 | --- | unaffected |
firefox55 | --- | unaffected |
People
(Reporter: marcia, Unassigned)
References
(Depends on 1 open bug)
Details
(Keywords: crash)
Crash Data
This bug was filed from the Socorro interface and is
report bp-02c5ca94-847a-4fc5-b3cc-2764a2170328.
=============================================================
This crash is new in 53 beta 5 and showed up on the explosiveness report: http://bit.ly/2mMvJsV
All the crashes seem to be confined to Mac OS and affects all versions of Mac OS.
Not sure which component this belongs in - I just made my best guess.
Comment 1•8 years ago
|
||
Does anyone know why this doesn't have a crash reason annotation?
Flags: needinfo?(erahm)
Reporter | ||
Comment 2•8 years ago
|
||
As phillip pointed out this has not been seen after Beta 5, so we can close it out if it doesn't reappear.
Comment 3•8 years ago
|
||
IPC was a reasonable start, but I looked at a half dozen of these and they all have stuff like mozilla::dom::PContentChild::Read(nsTArray<mozilla::dom::ServiceWorkerRegistrationData>*, IPC::Message const*, PickleIterator*) in the stack, so this is or was some service worker issue, I think.
status-firefox54:
--- → unaffected
status-firefox55:
--- → unaffected
Component: IPC → DOM: Service Workers
Comment 4•8 years ago
|
||
From the array, I'd guess this is the PContent InitServiceWorkers message.
Comment 5•8 years ago
|
||
(In reply to Benjamin Smedberg [:bsmedberg] from comment #1)
> Does anyone know why this doesn't have a crash reason annotation?
It looks like we only annotate in the parent process, but this was from the child process [1].
[1] https://hg.mozilla.org/releases/mozilla-beta/annotate/62467dd7218f/ipc/glue/ProtocolUtils.cpp#l302
Flags: needinfo?(erahm)
Comment 6•8 years ago
|
||
Ooh, we should fix that! That probably dates from the days where crash annotations in children were main-thread-only.
Updated•8 years ago
|
Blocks: ServiceWorkers-stability
Priority: -- → P3
Comment 7•7 years ago
|
||
(In reply to Eric Rahm [:erahm] (please no mozreview requests) from comment #5)
> (In reply to Benjamin Smedberg [:bsmedberg] from comment #1)
> > Does anyone know why this doesn't have a crash reason annotation?
>
> It looks like we only annotate in the parent process, but this was from the
> child process [1].
>
> [1]
> https://hg.mozilla.org/releases/mozilla-beta/annotate/62467dd7218f/ipc/glue/
> ProtocolUtils.cpp#l302
(In reply to Benjamin Smedberg from comment #6)
> Ooh, we should fix that! That probably dates from the days where crash
> annotations in children were main-thread-only.
Therefore, this bug Depends on:
Bug 1169641
AnnoteCrashReporter like API so that child processes annotations show up in the parent reports
NEW Unassigned (NEW bug with no priority)
Reported: 3 years ago
Modified: 3 years ago
(In reply to Milan Sreckovic [:milan] from comment #0)
> From a side conversation with :ted.mielczarek and :jimm :
>
> When you call AnnotateCrashReport in a child process the annotation is sent
> via IPC to the parent process, but it's not stored in the global annotations
> hashtable there, there's a hashtable that hangs off of
> PCrashReporterParent(?) that holds them, and when the child crashes it
> provides them to the crashreporter code to write out the .extra file. When
> the parent crashes it just writes out the global hashtable.
>
> ... we could special-case certain annotations to make their way into the
> parent's global hashtable, or we can add an extra API for this (we have
> special APIs for other kinds of annotations).
Depends on: 1169641
Comment 8•6 years ago
|
||
Closing because no crashes reported for 12 weeks.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•