1375503 - Crash in shutdownhang | NtWaitForKeyedEvent | RtlSleepConditionVariableSRW | mozilla::MainThreadIdlePeriod::GetIdlePeriodHint

Status: RESOLVED WONTFIX

(Core :: DOM: Core & HTML, defect)

Description

[[:philipp]]

This bug was filed from the Socorro interface and is 
report bp-eceb7ea8-ba85-4c47-b685-b72660170622.
=============================================================
Crashing Thread (0)
Frame 	Module 	Signature 	Source
0 	ntdll.dll 	KiFastSystemCallRet 	
1 	ntdll.dll 	NtWaitForKeyedEvent 	
2 	ntdll.dll 	RtlSleepConditionVariableSRW 	
3 	xul.dll 	mozilla::MainThreadIdlePeriod::GetIdlePeriodHint(mozilla::TimeStamp*) 	xpcom/threads/MainThreadIdlePeriod.cpp:40
4 	nss3.dll 	_PR_MD_SET_CURRENT_THREAD_NAME 	nsprpub/pr/src/md/windows/w95thred.c:197

these shutdownhang reports seem to be regressing in firefox 55 - they are all from 32bit clients on windows 7 and crashes in a codepath last touched in bug 1311425.
so far they account for 2.6% of browser crashes in 55.0b3

Comment 1

[Marco Castelluccio [:marco]]

For some reason this is happening often to pt-BR users.

Comment 2

[Liz Henry (:lizzard) (relman/hg->git project)]

Andres, might this be related to bug 1311425?

Comment 3

[Andreas Farre [:farre]]

It might be, but that stack seems very strange to me, not sure how GetIdlePeriodHint ended up in there. To get to MainThreadIdlePeriod::GetIdlePeriodHint the stack should've been:

#0  0x00007f93b83a1c57 in mozilla::MainThreadIdlePeriod::GetIdlePeriodHint(mozilla::TimeStamp*) (this=0x7f93b5446cc0, aIdleDeadline=0x7ffda3bc9088)
    at /home/farre/src/gecko/work-1/xpcom/threads/MainThreadIdlePeriod.cpp:30
#1  0x00007f93b83b05c8 in nsThread::GetIdleDeadline() (this=0x7f93c849eec0) at /home/farre/src/gecko/work-1/xpcom/threads/nsThread.cpp:1074
#2  0x00007f93b83b06b0 in nsThread::HasPendingEvents(bool*) (this=0x7f93c849eec0, aResult=0x7ffda3bc90ef) at /home/farre/src/gecko/work-1/xpcom/threads/nsThread.cpp:1116
#3  0x00007f93b83b2a2c in hasPendingEvents(nsIThread*) (aThread=0x7f93c8500008) at /home/farre/src/gecko/work-1/xpcom/threads/nsThreadUtils.cpp:432
#4  0x00007f93b83b2a2c in NS_HasPendingEvents(nsIThread*) (aThread=0x7f93c8500008) at /home/farre/src/gecko/work-1/xpcom/threads/nsThreadUtils.cpp:450
#5  0x00007f93ba745629 in nsBaseAppShell::OnProcessNextEvent(nsIThreadInternal*, bool) (this=0x7f93b0d8de80, thr=0x7f93c849eec0, mayWait=<optimized out>)
    at /home/farre/src/gecko/work-1/widget/nsBaseAppShell.cpp:281
#6  0x00007f93ba745810 in non-virtual thunk to nsBaseAppShell::OnProcessNextEvent(nsIThreadInternal*, bool) () at /home/farre/src/gecko/work-1/obj-linux-release/dist/include/mozilla/ipc/ProtocolUtils.h:180
#7  0x00007f93b83b0ca2 in nsThread::ProcessNextEvent(bool, bool*) (this=0x7f93c849eec0, aMayWait=<optimized out>, aResult=0x7ffda3bc9287) at /home/farre/src/gecko/work-1/xpcom/threads/nsThread.cpp:1360
#8  0x00007f93b83b2a6f in NS_ProcessNextEvent(nsIThread*, bool) (aThread=0x7f93c8500008, aMayWait=<error reading variable: access outside bounds of object referenced via synthetic pointer>)
    at /home/farre/src/gecko/work-1/xpcom/threads/nsThreadUtils.cpp:474
#9  0x00007f93b88c5ac8 in mozilla::ipc::MessagePump::Run(base::MessagePump::Delegate*) (this=0x7f93c84de330, aDelegate=0x7ffda3bc93a8) at /home/farre/src/gecko/work-1/ipc/glue/MessagePump.cpp:97
#10 0x00007f93b88618eb in MessageLoop::RunInternal() (this=0x7fffffffffffffff) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:320
#11 0x00007f93b88618eb in MessageLoop::RunHandler() (this=<optimized out>) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:313
#12 0x00007f93b88618eb in MessageLoop::Run() (this=0x7fffffffffffffff) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:293
#13 0x00007f93ba745359 in nsBaseAppShell::Run() (this=0x7f93b0d8de80) at /home/farre/src/gecko/work-1/widget/nsBaseAppShell.cpp:156
#14 0x00007f93bbe13d74 in XRE_RunAppShell() () at /home/farre/src/gecko/work-1/toolkit/xre/nsEmbedFunctions.cpp:895
#15 0x00007f93b88618eb in MessageLoop::RunInternal() (this=0x7fffffffffffffff) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:320
#16 0x00007f93b88618eb in MessageLoop::RunHandler() (this=<optimized out>) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:313
#17 0x00007f93b88618eb in MessageLoop::Run() (this=0x7fffffffffffffff) at /home/farre/src/gecko/work-1/ipc/chromium/src/base/message_loop.cc:293
#18 0x00007f93bbe13a8b in XRE_InitChildProcess(int, char**, XREChildData const*) (aArgc=<optimized out>, aArgv=0x7ffda3bca7b8, aChildData=<optimized out>)
    at /home/farre/src/gecko/work-1/toolkit/xre/nsEmbedFunctions.cpp:711
#19 0x0000000000405991 in content_process_main(mozilla::Bootstrap*, int, char**) (bootstrap=0x7f93c84bf0a8, argc=<optimized out>, argv=0x7ffda3bca7b8)
    at /home/farre/src/gecko/work-1/browser/app/../../ipc/contentproc/plugin-container.cpp:64
#20 0x0000000000405991 in main(int, char**, char**) (argc=<optimized out>, argv=0x7ffda3bca7b8, envp=0x7ffda3bca840) at /home/farre/src/gecko/work-1/browser/app/nsBrowserApp.cpp:286

Comment 4

[[:philipp]]

this was a generic signature for shutdownhangs & it has shifted now. i'll file a followup bug for the increase of shutdownhangs for pt-br users...