New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 630011 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner: ----
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug



Sign in to add a comment

InsertHTML command with display:flex crashes

Project Member Reported by ClusterFuzz, Jul 20 2016

Issue description

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4624105215361024

Fuzzer: ochang_domfuzzer
Job Type: linux_asan_chrome_v8_arm
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00000000
Crash State:
  blink::hasEditableStyle
  blink::Node::hasEditableStyle
  blink::CompositeEditCommand::insertNodeBefore
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=406399:406435

Minimized Testcase (9.66 Kb): https://cluster-fuzz.appspot.com/download/AMIfv948-ZSDcgLPHvhKQNY9uSUza81szdIzzj65Lw1sMtSySqSMF_kK7j73QgP-3m_wTFoT2BryIXn2inVBgEeDlrvQTtzG_Ejd3gAMSndU5pBDf1XaL4oo9NUutwIkMCbuGUyvw_epUMlfrW-lvBf5ypsNRMchfQ?testcase_id=4624105215361024

Filer: mummareddy

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Components: Blink>DOM
Labels: M-54 Te-Logged
Owner: yoichio@chromium.org
Status: Assigned (was: Untriaged)

Author: yoichio
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/0793f8f201f0a8468e6e3dc5b1a085e4161a684d
Time: Tue Jul 19 23:28:56 2016
Lines 557, 604-620 of file Node.cpp which potentially caused crash are changed in this cl (frame #1, "blink::hasEditableStyle"; frame #2, "blink::Node::hasEditableStyle").
Minimum distance from crash line to modified line: 0. (file: Node.cpp, crashed on: 554, modified: 554).

Suspected Project: chromium
Suspected Component: Blink>DOM
Project Member

Comment 2 by ClusterFuzz, Jul 20 2016

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=5264724593999872

Fuzzer: inferno_layout_test_unmodified
Job Type: windows_syzyasan_chrome
Platform Id: windows

Crash Type: UNKNOWN
Crash Address: 0x00000003
Crash State:
  blink::hasEditableStyle
  blink::Node::hasEditableStyle
  blink::CompositeEditCommand::insertNodeBefore
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_chrome&range=406435:406472

Minimized Testcase (0.37 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97ItagkRnRd0CY3Yh4Jdas3e9-BFp2UAwrICHhDPKfMOpo9_2CBeW25tQnBfu1hpbNy5ZtgELYDyFv6hna8Haa25rbCJnp1o0fF25oktAY6HlXtdS_fRFPOUDMTWjUznXDVUJdUW27CRwslGP0ye3nbPYwIeA?testcase_id=5264724593999872

Filer: mummareddy

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.

Comment 3 by tkent@chromium.org, Jul 21 2016

Components: -Blink>DOM Blink>Editing
Labels: -ClusterFuzz Clusterfuzz

Comment 4 by yosin@chromium.org, Jul 21 2016

Components: -Blink>Editing Blink>Editing>Command
Labels: -Pri-1 OS-Windows Pri-2
Owner: ----
Status: Available (was: Assigned)
Summary: InsertHTML command with display:flex crashes (was: Crash in blink::hasEditableStyle)
Lower to Pri-2, since real world usage of insertHTML command with display:flex is low.

DOM tree at nullptr reference:

m_endingSelection.showTreeForThis()
BODY	000003C635B83290
	DIV	000003C635B832F8 ID="test" (editable) (focused)
		#text	000003C635B837C8 "\n   x"
		DIV	000003C635B83588 (editable)
			#text	000003C635B835F0 "foo"
		DIV	000003C635B836A8 (editable)
			#text	000003C635B83710 "bar"
SE		#text	000003C635B83360 "x\n  "
	#text	000003C635B833B0 "\n"
	SCRIPT	000003C635B83400
		#text	000003C635B83478 "...script..."
<void>
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 6 by ClusterFuzz, Nov 24 2016

ClusterFuzz has detected this issue as fixed in range 434178:434216.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=4624105215361024

Fuzzer: ochang_domfuzzer
Job Type: linux_asan_chrome_v8_arm
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00000000
Crash State:
  blink::hasEditableStyle
  blink::Node::hasEditableStyle
  blink::CompositeEditCommand::insertNodeBefore
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=406399:406435
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_chrome_v8_arm&range=434178:434216

Minimized Testcase (9.66 Kb): https://cluster-fuzz.appspot.com/download/AMIfv948-ZSDcgLPHvhKQNY9uSUza81szdIzzj65Lw1sMtSySqSMF_kK7j73QgP-3m_wTFoT2BryIXn2inVBgEeDlrvQTtzG_Ejd3gAMSndU5pBDf1XaL4oo9NUutwIkMCbuGUyvw_epUMlfrW-lvBf5ypsNRMchfQ?testcase_id=4624105215361024

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
Status: Fixed (was: Available)
Changing the status to Fixed as per Comment# 6, as Cluster Fuxx detected this issue to be fixed.
Thanks.

Sign in to add a comment