Closed Bug 1547550 Opened 6 years ago Closed 6 years ago

Crash in [@ PR_JoinThread | nsThreadShutdownAckEvent::Run]

Categories

(Core :: General, defect)

Product:
Core
Component:
General
Version:
67 Branch
Platform:
Unspecified
Windows 7
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

()

Status:
RESOLVED DUPLICATE of bug 1535301
Tracking Flags:
Tracking Status
firefox66 --- wontfix
firefox67 --- wontfix
firefox67.0.1 --- wontfix
firefox68 --- wontfix

People

(Reporter: pascalc, Unassigned)

Details

(Keywords: crash)

Crash Data

This bug is for crash report bp-8e19a112-430d-4dcb-9c49-a71d40190428.

Top 10 frames of crashing thread:

0 nss3.dll PR_WaitCondVar nsprpub/pr/src/threads/combined/prucv.c:523
1 nss3.dll PR_JoinThread nsprpub/pr/src/threads/combined/pruthr.c:1533
2 xul.dll nsresult nsThreadShutdownAckEvent::Run xpcom/threads/nsThread.cpp:265
3 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1179
4 xul.dll NS_ProcessNextEvent xpcom/threads/nsThreadUtils.cpp:482
5 xul.dll static bool mozilla::SpinEventLoopUntil<mozilla::ProcessFailureBehavior::ReportToCaller, `lambda at z:/task_1556111056/build/src/dom/quota/ActorsParent.cpp:2404:5'> xpcom/threads/nsThreadUtils.h:348
6 xul.dll nsresult mozilla::dom::quota::QuotaManager::Observer::Observe dom/quota/ActorsParent.cpp:2404
7 xul.dll nsObserverList::NotifyObservers xpcom/ds/nsObserverList.cpp:66
8 xul.dll nsObserverService::NotifyObservers xpcom/ds/nsObserverService.cpp:293
9 xul.dll nsXREDirProvider::DoShutdown toolkit/xre/nsXREDirProvider.cpp:1094

We had a handful of these crahes per beta but they started spiking in beta 14 yesterday. Marcia, Calixte, would it be possible to see if a patch uplifted in beta 14 is the cause? Maybe with clouseau? Thanks

https://hg.mozilla.org/releases/mozilla-beta/pushloghtml?fromchange=FIREFOX_67_0b13_RELEASE&tochange=FIREFOX_67_0b14_RELEASE

Tracking because of the spike

status-firefox66: --- → affected
status-firefox67: --- → affected
status-firefox68: --- → affected
tracking-firefox67: --- → +

Alastor, could it have been caused by your patch in https://hg.mozilla.org/releases/mozilla-beta/annotate/151cf24aa0fe45af946282b275e30deb908af841/nsprpub/pr/src/threads/combined/prucv.c ?

(Also needindo padenot for timezone reasons in case we can have a quick investigation and a patch pour our last betas)

Flags: needinfo?(padenot)
Flags: needinfo?(alwu)

Nothing with clouseau, but according to the backtrace and the pushlog, I'd say that maybe patch 81b2e6c7139d for bug 1546500 is the culprit here.
:kmag, could you have a look please ?

Flags: needinfo?(kmaglione+bmo)

Not us, but maybe Jan can have a look as well, it's the Quota Manager is in the stack.

Flags: needinfo?(padenot)
Flags: needinfo?(jvarga)
Flags: needinfo?(alwu)

QuotaManager is in the stack trace, but it spins a nested event loop there and AFAIK we haven't landed anything on beta related to quota manager that could cause the crash.

Flags: needinfo?(jvarga)
Group: core-security
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(kmaglione+bmo)
Resolution: --- → DUPLICATE
tracking-firefox67: + → ---

Marking as wontfix for our releases in flight to get it out of some triage queries as this is a dupe of bug 1535301 where the work will happen.

status-firefox67: affected → wontfix
status-firefox67.0.1: --- → wontfix
status-firefox68: affected → wontfix
Group: core-security
You need to log in before you can comment on or make changes to this bug.