Closed Bug 1346437 Opened 7 years ago Closed 7 years ago

Crash in mozilla::ipc::MessageChannel::AssertWorkerThread | mozilla::ipc::MessageChannel::Send | mozilla::dom::asmjscache::PAsmJSCacheEntryParent::SendOnOpenMetadataForRead

Categories

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

52 Branch
x86
Windows
defect

Tracking

()

RESOLVED DUPLICATE of bug 1331209
Tracking Status
firefox-esr45 --- unaffected
firefox52 --- wontfix
firefox-esr52 --- affected
firefox53 --- wontfix
firefox54 --- wontfix
firefox55 --- wontfix
firefox56 --- wontfix
firefox57 --- fix-optional
firefox58 --- fix-optional

People

(Reporter: philipp, Unassigned)

References

Details

(Keywords: crash, regression)

Crash Data

This bug was filed from the Socorro interface and is 
report bp-74d6737f-ed9a-4ff1-802e-4e05a2170310.
=============================================================
Crashing Thread (28)
Frame 	Module 	Signature 	Source
0 	xul.dll 	mozilla::ipc::MessageChannel::AssertWorkerThread() 	obj-firefox/dist/include/mozilla/ipc/MessageChannel.h:438
1 	xul.dll 	mozilla::ipc::MessageChannel::Send(IPC::Message*) 	ipc/glue/MessageChannel.cpp:776
2 	xul.dll 	mozilla::dom::asmjscache::PAsmJSCacheEntryParent::SendOnOpenMetadataForRead(mozilla::dom::asmjscache::Metadata const&) 	obj-firefox/ipc/ipdl/PAsmJSCacheEntryParent.cpp:66
3 	xul.dll 	mozilla::dom::asmjscache::`anonymous namespace'::ParentRunnable::Run 	dom/asmjscache/AsmJSCache.cpp:1003
4 	xul.dll 	nsThread::ProcessNextEvent(bool, bool*) 	xpcom/threads/nsThread.cpp:1216
5 	xul.dll 	NS_ProcessNextEvent(nsIThread*, bool) 	xpcom/glue/nsThreadUtils.cpp:361
6 	xul.dll 	nsThread::Shutdown() 	xpcom/threads/nsThread.cpp:983
7 	xul.dll 	mozilla::LazyIdleThread::ShutdownThread() 	xpcom/threads/LazyIdleThread.cpp:310
8 	xul.dll 	mozilla::LazyIdleThread::Shutdown() 	xpcom/threads/LazyIdleThread.cpp:489
9 	xul.dll 	mozilla::dom::quota::QuotaManager::Shutdown() 	dom/quota/ActorsParent.cpp:3206
10 	xul.dll 	mozilla::dom::quota::QuotaManager::ShutdownRunnable::Run() 	dom/quota/ActorsParent.cpp:2406
11 	xul.dll 	nsThread::ProcessNextEvent(bool, bool*) 	xpcom/threads/nsThread.cpp:1216
12 	xul.dll 	NS_ProcessNextEvent(nsIThread*, bool) 	xpcom/glue/nsThreadUtils.cpp:361
13 	xul.dll 	mozilla::ipc::MessagePumpForNonMainThreads::Run(base::MessagePump::Delegate*) 	ipc/glue/MessagePump.cpp:368
14 	xul.dll 	MessageLoop::RunHandler() 	ipc/chromium/src/base/message_loop.cc:225
15 	xul.dll 	MessageLoop::Run() 	ipc/chromium/src/base/message_loop.cc:205
16 	xul.dll 	nsThread::ThreadFunc(void*) 	xpcom/threads/nsThread.cpp:467
17 	nss3.dll 	_PR_NativeRunThread 	nsprpub/pr/src/threads/combined/pruthr.c:397
18 	nss3.dll 	pr_root 	nsprpub/pr/src/md/windows/w95thred.c:95
19 	ucrtbase.dll 	_o__CIpow 	
20 	kernel32.dll 	BaseThreadInitThunk 	
21 	ntdll.dll 	__RtlUserThreadStart 	
22 	ntdll.dll 	_RtlUserThreadStart

this crash signature is regressing with firefox 52 - so far reports are only coming in form 32bit versions on windows.
Flags: needinfo?(luke)
Jan: can we just relax the assert here?
Flags: needinfo?(luke) → needinfo?(jvarga)
Crash Signature: [@ mozilla::ipc::MessageChannel::AssertWorkerThread | mozilla::ipc::MessageChannel::Send | mozilla::dom::asmjscache::PAsmJSCacheEntryParent::SendOnOpenMetadataForRead] → [@ mozilla::ipc::MessageChannel::AssertWorkerThread | mozilla::ipc::MessageChannel::Send | mozilla::dom::asmjscache::PAsmJSCacheEntryParent::SendOnOpenMetadataForRead] [@ mozilla::ipc::MessageChannel::AssertWorkerThread | mozilla::ipc::MessageChannel::Cxx…
Given that no one is assigned and we are about to release 53, wontfix.
Andrew, this is a pretty low volume crash but since it seems to be a recent regression, is it worth investigating?
Jan and I spoke about this and he's going to comment here.
Flags: needinfo?(overholt)
any updates here? the user comment at bp-27a72e81-97f6-422a-a9b1-941320170802 says they are crashing at mega.nz...
I think this bug has the same underlying cause as bug 1331209, where I have a patch up for review.
Marking fix optional so that we can exclude this from triage. This isn't fixed yet but we could potentially still take a patch for 57/58.
Given comment 6, marking P2 to match bug 1331209.
Priority: -- → P2
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(jvarga)
Resolution: --- → DUPLICATE
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.