1727500 - Crash in [@ PossiblyCrash]

Status: RESOLVED FIXED

(Core :: Security, defect)

Description

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

The code added in bug 1723204 already catches issues, 8 parent process crashes from 5 installations so far. Please check the crash reports and if the change will be reverted because it is a frequent crash.

Crash report: https://crash-stats.mozilla.org/report/index/e181a367-e059-45ab-8bdc-19f3d0210825

MOZ_CRASH Reason: (None)

Top 10 frames of crashing thread:

0 xul.dll PossiblyCrash dom/security/nsContentSecurityUtils.cpp:518
1 xul.dll static nsContentSecurityUtils::ValidateScriptFilename dom/security/nsContentSecurityUtils.cpp:1238
2 xul.dll static js::ScriptSourceObject::initFromOptions js/src/vm/JSScript.cpp:1764
3 xul.dll static js::frontend::CompilationStencil::instantiateStencilAfterPreparation js/src/frontend/Stencil.cpp:1510
4 xul.dll js::frontend::InstantiateStencils js/src/frontend/BytecodeCompiler.cpp:386
5 xul.dll CompileGlobalScriptToStencilAndMaybeInstantiate<char16_t> js/src/frontend/BytecodeCompiler.cpp:313
6 xul.dll js::frontend::CompileGlobalScript js/src/frontend/BytecodeCompiler.cpp:434
7 xul.dll nsMessageManagerScriptExecutor::LoadScriptInternal dom/base/nsFrameMessageManager.cpp:1187
8 xul.dll mozilla::dom::ContentProcessMessageManager::LoadScript dom/base/ContentProcessMessageManager.cpp:119
9 xul.dll SameParentProcessMessageManagerCallback::DoLoadMessageManagerScript dom/base/nsFrameMessageManager.cpp:1398

Comment 1

[Tom Ritter [:tjr]]

Attachment: Bug 1727500: Disable crash reporting now that we have some data r?freddyb

Comment 2

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/3eaabde171ef

Comment 3

[Tom Ritter [:tjr]]

Based on install times of the submitted reports; it seems like we successfully did not crash the same user twice.