Closed Bug 1367281 Opened 7 years ago Closed 7 years ago

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)

Categories

(Toolkit :: Telemetry, defect, P1)

Product:
Toolkit
Component:
Telemetry
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla55
Tracking Flags:
Tracking Status
firefox-esr52 --- unaffected
firefox53 --- unaffected
firefox54 --- unaffected
firefox55 --- fixed

People

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

References

(Blocks 1 open bug)

Details

(Keywords: assertion, crash, intermittent-failure)

Crash Data

Attachments

(1 file)

Wait on member Mutexes in TelemetryImpl destructor
(deleted), patch
chutten
: review+
Details | Diff | Splinter Review
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]
Blocks: 1367344
Blocks: 1277552
Assignee: nobody → gfritzsche
Status: NEW → ASSIGNED
Priority: -- → P1
Attachment #8870749 - Flags: review?(chutten)
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+
Keywords: checkin-needed
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
https://hg.mozilla.org/mozilla-central/rev/b927981cceb7
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: