Intermittent test_ext_contentscript_xrays.js | application crashed [@ `anonymous namespace'::TelemetryImpl::~TelemetryImpl] (Assertion failure: !mOwningThread (destroying a still-owned lock!), at mozilla/Mutex.h:57)

RESOLVED FIXED in Firefox 55

Status

()

P1
critical
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: intermittent-bug-filer, Assigned: gfritzsche)

Tracking

(Blocks: 2 bugs, {assertion, crash, intermittent-failure})

unspecified
mozilla55
assertion, crash, intermittent-failure
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55 fixed)

Details

(crash signature)

Attachments

(1 attachment)

Keywords: assertion
(Assignee)

Comment 1

a year ago
Relevant bits:
> PROCESS-CRASH | xpcshell-remote.ini:toolkit/components/extensions/test/xpcshell/test_ext_contentscript_xrays.js | application crashed [@ `anonymous namespace'::TelemetryImpl::~TelemetryImpl]
> ...
> Assertion failure: !mOwningThread (destroying a still-owned lock!), at c:\\builds\\moz2_slave\\autoland-w64-d-000000000000000\\build\\src\\obj-firefox\\dist\\include\\mozilla/Mutex.h:57

Crash site:
https://hg.mozilla.org/integration/autoland/file/d4dc4bfab1eeb363de9202cd99f44fdf862f28ea/toolkit/components/telemetry/Telemetry.cpp#l1266

Crash stack:
> 0  xul.dll!`anonymous namespace'::TelemetryImpl::~TelemetryImpl [Telemetry.cpp:d4dc4bfab1ee : 1266 + 0x1c1]
> 1  xul.dll!`anonymous namespace'::TelemetryImpl::Release [Telemetry.cpp:d4dc4bfab1ee : 2723 + 0x52]
> 2  xul.dll!`anonymous namespace'::TelemetryImpl::ShutdownTelemetry [Telemetry.cpp:d4dc4bfab1ee : 2362 + 0x11]
> 3  xul.dll!nsComponentManagerImpl::KnownModule::`scalar deleting destructor'(unsigned int) + 0x28
> 4  xul.dll!nsTArray_Impl<nsAutoPtr<nsComponentManagerImpl::KnownModule>,nsTArrayInfallibleAllocator>::RemoveElementsAt(unsigned __int64,unsigned __int64) [nsTArray.h:d4dc4bfab1ee : 2055 + 0x20]
> 5  xul.dll!nsComponentManagerImpl::Shutdown() [nsComponentManager.cpp:d4dc4bfab1ee : 821 + 0x14]
> 6  xul.dll!mozilla::ShutdownXPCOM(nsIServiceManager *) [XPCOMInit.cpp:d4dc4bfab1ee : 1007 + 0x5]
> 7  xul.dll!XRE_TermEmbedding() [nsEmbedFunctions.cpp:d4dc4bfab1ee : 225 + 0x7]
> 8  xul.dll!mozilla::ipc::ScopedXREEmbed::Stop() [ScopedXREEmbed.cpp:d4dc4bfab1ee : 117 + 0x5]
> 9  xul.dll!XRE_InitChildProcess(int,char * * const,XREChildData const *) [nsEmbedFunctions.cpp:d4dc4bfab1ee : 713 + 0x9]
> 10  plugin-container.exe!content_process_main(mozilla::Bootstrap *,int,char * * const) [plugin-container.cpp:d4dc4bfab1ee : 64 + 0x13]
(Assignee)

Updated

a year ago
Blocks: 1367344
(Assignee)

Updated

a year ago
Blocks: 1277552
(Assignee)

Comment 3

a year ago
Created attachment 8870749 [details] [diff] [review]
Wait on member Mutexes in TelemetryImpl destructor
(Assignee)

Updated

a year ago
Assignee: nobody → gfritzsche
Status: NEW → ASSIGNED
(Assignee)

Updated

a year ago
Priority: -- → P1
(Assignee)

Updated

a year ago
Attachment #8870749 - Flags: review?(chutten)

Comment 5

a year ago
Comment on attachment 8870749 [details] [diff] [review]
Wait on member Mutexes in TelemetryImpl destructor

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

Seems legit to me.
Attachment #8870749 - Flags: review?(chutten) → review+
(Assignee)

Updated

a year ago
Keywords: checkin-needed

Comment 6

a year ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b927981cceb7
Wait on member Mutexes in TelemetryImpl destructor. r=chutten
Keywords: checkin-needed

Comment 7

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/b927981cceb7
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
status-firefox53: --- → unaffected
status-firefox54: --- → unaffected
status-firefox-esr52: --- → unaffected
You need to log in before you can comment on or make changes to this bug.