1449596 - Assertion failure: aElement && aProperty, at /home/worker/workspace/build/src/editor/libeditor/CSSEditUtils.cpp:899

Status: RESOLVED WORKSFORME

(Core :: DOM: Editor, defect, P3)

Description

[Jason Kratzer [:jkratzer]]

Attachment: testcase

Testcase found while fuzzing esr52 rev d61516b059c1.

rax = 0x0000000000625d50   rdx = 0x0000000000000000
rcx = 0x00007fa6c32d52dd   rbx = 0x00007fa6c87054e1
rsi = 0x00007fa6c35a4770   rdi = 0x00007fa6c35a3540
rbp = 0x00007ffde0a78880   rsp = 0x00007ffde0a78840
r8 = 0x00007fa6c35a4770    r9 = 0x00007fa6cab59c00
r10 = 0x0000000000000043   r11 = 0x0000000000000000
r12 = 0x00007ffde0a78918   r13 = 0x00007fa6a507d800
r14 = 0x00007ffde0a78900   r15 = 0x00007ffde0a78908
rip = 0x00007fa6c69a6c1c
OS|Linux|0.0.0 Linux 4.4.0-116-generic #140-Ubuntu SMP Mon Feb 12 21:23:04 UTC 2018 x86_64
CPU|amd64|family 6 model 78 stepping 3|1
GPU|||
Crash|SIGSEGV|0x0|0
0|0|libxul.so|mozilla::CSSEditUtils::SetCSSEquivalentToHTMLStyle|hg:hg.mozilla.org/releases/mozilla-esr52:editor/libeditor/CSSEditUtils.cpp:d61516b059c1|899|0x0
0|1|libxul.so|mozilla::HTMLEditRules::WillAlign|hg:hg.mozilla.org/releases/mozilla-esr52:editor/libeditor/HTMLEditRules.cpp:d61516b059c1|4667|0x26
0|2|libxul.so|mozilla::HTMLEditRules::WillDoAction|hg:hg.mozilla.org/releases/mozilla-esr52:editor/libeditor/HTMLEditRules.cpp:d61516b059c1|632|0x12
0|3|libxul.so|mozilla::HTMLEditor::Align|hg:hg.mozilla.org/releases/mozilla-esr52:editor/libeditor/HTMLEditor.cpp:d61516b059c1|2266|0x27
0|4|libxul.so|nsAlignCommand::SetState|hg:hg.mozilla.org/releases/mozilla-esr52:editor/composer/nsComposerCommands.cpp:d61516b059c1|962|0x17
0|5|libxul.so|nsMultiStateCommand::DoCommandParams|hg:hg.mozilla.org/releases/mozilla-esr52:editor/composer/nsComposerCommands.cpp:d61516b059c1|595|0x13
0|6|libxul.so|nsControllerCommandTable::DoCommandParams|hg:hg.mozilla.org/releases/mozilla-esr52:embedding/components/commandhandler/nsControllerCommandTable.cpp:d61516b059c1|162|0x20
0|7|libxul.so|nsBaseCommandController::DoCommandWithParams|hg:hg.mozilla.org/releases/mozilla-esr52:embedding/components/commandhandler/nsBaseCommandController.cpp:d61516b059c1|152|0x1c
0|8|libxul.so|nsCommandManager::DoCommand|hg:hg.mozilla.org/releases/mozilla-esr52:embedding/components/commandhandler/nsCommandManager.cpp:d61516b059c1|212|0xc
0|9|libxul.so|nsHTMLDocument::ExecCommand|hg:hg.mozilla.org/releases/mozilla-esr52:dom/html/nsHTMLDocument.cpp:d61516b059c1|3257|0x23
0|10|libxul.so|mozilla::dom::HTMLDocumentBinding::execCommand|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dom/bindings/HTMLDocumentBinding.cpp:d61516b059c1|829|0x20
0|11|libxul.so|mozilla::dom::GenericBindingMethod|hg:hg.mozilla.org/releases/mozilla-esr52:dom/bindings/BindingUtils.cpp:d61516b059c1|2904|0x9
0|12|libxul.so|js::CallJSNative|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jscntxtinlines.h:d61516b059c1|239|0x9
0|13|libxul.so|js::InternalCallOrConstruct|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|459|0xf
0|14|libxul.so|js::Call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|523|0x5
0|15|libxul.so|js::Wrapper::call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/proxy/Wrapper.cpp:d61516b059c1|165|0xb
0|16|libxul.so|js::CrossCompartmentWrapper::call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/proxy/CrossCompartmentWrapper.cpp:d61516b059c1|333|0x15
0|17|libxul.so|js::Proxy::call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/proxy/Proxy.cpp:d61516b059c1|400|0x15
0|18|libxul.so|js::proxy_Call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/proxy/Proxy.cpp:d61516b059c1|689|0x8
0|19|libxul.so|js::CallJSNative|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jscntxtinlines.h:d61516b059c1|239|0x9
0|20|libxul.so|js::InternalCallOrConstruct|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|447|0xe
0|21|libxul.so|Interpret|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|510|0xf
0|22|libxul.so|js::RunScript|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|405|0xb
0|23|libxul.so|js::ExecuteKernel|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|686|0x5
0|24|libxul.so|js::Execute|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|719|0x28
0|25|libxul.so|Evaluate|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jsapi.cpp:d61516b059c1|4440|0xf
0|26|libxul.so|Evaluate|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jsapi.cpp:d61516b059c1|4466|0x1d
0|27|libxul.so|nsJSUtils::EvaluateString|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsJSUtils.cpp:d61516b059c1|207|0x1c
0|28|libxul.so|nsJSUtils::EvaluateString|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsJSUtils.cpp:d61516b059c1|275|0x2a
0|29|libxul.so|nsScriptLoader::EvaluateScript|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsScriptLoader.cpp:d61516b059c1|2194|0x21
0|30|libxul.so|nsScriptLoader::ProcessRequest|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsScriptLoader.cpp:d61516b059c1|1979|0xb
0|31|libxul.so|nsScriptLoader::ProcessScriptElement|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsScriptLoader.cpp:d61516b059c1|1712|0xf
0|32|libxul.so|nsScriptElement::MaybeProcessScript|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsScriptElement.cpp:d61516b059c1|149|0x13
0|33|libxul.so|nsIScriptElement::AttemptToExecute|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsIScriptElement.h:d61516b059c1|222|0x3
0|34|libxul.so|nsHtml5TreeOpExecutor::RunScript|hg:hg.mozilla.org/releases/mozilla-esr52:parser/html/nsHtml5TreeOpExecutor.cpp:d61516b059c1|666|0x10
0|35|libxul.so|nsHtml5TreeOpExecutor::RunFlushLoop|hg:hg.mozilla.org/releases/mozilla-esr52:parser/html/nsHtml5TreeOpExecutor.cpp:d61516b059c1|489|0x8
0|36|libxul.so|nsHtml5ExecutorFlusher::Run|hg:hg.mozilla.org/releases/mozilla-esr52:parser/html/nsHtml5StreamParser.cpp:d61516b059c1|128|0x8
0|37|libxul.so|nsThread::ProcessNextEvent|hg:hg.mozilla.org/releases/mozilla-esr52:xpcom/threads/nsThread.cpp:d61516b059c1|1216|0x11
0|38|libxul.so|NS_ProcessNextEvent|hg:hg.mozilla.org/releases/mozilla-esr52:xpcom/glue/nsThreadUtils.cpp:d61516b059c1|361|0xd
0|39|libxul.so|mozilla::ipc::MessagePump::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/glue/MessagePump.cpp:d61516b059c1|124|0xd
0|40|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|232|0x17
0|41|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|225|0x8
0|42|libxul.so|nsBaseAppShell::Run|hg:hg.mozilla.org/releases/mozilla-esr52:widget/nsBaseAppShell.cpp:d61516b059c1|156|0xd
0|43|libxul.so|XRE_RunAppShell|hg:hg.mozilla.org/releases/mozilla-esr52:toolkit/xre/nsEmbedFunctions.cpp:d61516b059c1|866|0x6
0|44|libxul.so|mozilla::ipc::MessagePumpForChildProcess::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/glue/MessagePump.cpp:d61516b059c1|269|0x5
0|45|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|232|0x17
0|46|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|225|0x8
0|47|libxul.so|XRE_InitChildProcess|hg:hg.mozilla.org/releases/mozilla-esr52:toolkit/xre/nsEmbedFunctions.cpp:d61516b059c1|698|0xf
0|48|plugin-container|content_process_main|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/contentproc/plugin-container.cpp:d61516b059c1|197|0xe
0|49|libc-2.23.so||||0x20830
0|50|plugin-container|MOZ_ReportAssertionFailure|hg:hg.mozilla.org/releases/mozilla-esr52:mfbt/Assertions.h:d61516b059c1|170|0x5

Comment 1

[Bugmon [:jkratzer for issues]]

Bugmon Analysis:
Unable to reproduce bug using the following builds:

mozilla-central 20210224100119-b3eb91f0b5a7
mozilla-central 20200226092757-7f41334e1044

Comment 2

[Jens Stutte [:jstutte]]

This seems to have been fixed somewhere more than a year ago.