New issue
Advanced search Search tips

Issue 615800 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner: ----
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug-Regression



Sign in to add a comment

FormatBlock command crashes with unusual HTML

Project Member Reported by ClusterFuzz, May 30 2016

Issue description

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_lsan_chrome_mp
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000030
Crash State:
  blink::CompositeEditCommand::cloneParagraphUnderNewElement
  blink::CompositeEditCommand::moveParagraphWithClones
  blink::FormatBlockCommand::formatRange
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=144946:145047

Minimized Testcase (1.26 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96lHvTU6UwxYz26gU3pBv9KyWJSp6FD4ua_QDc4cxvX2B6kxIgC9QLVyunPTsVYnHTYd6OTCaobyUD2tRroN5pjyZ4l_Rxc6dyWLIDiPEFIe_1vMKUktuvlvCNLodWXs8q9gYfg9dh2zeE_sASZuxqJYPKK6Q

Filer: ajha

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

Comment 1 by ajha@chromium.org, May 30 2016

Cc: yosin@chromium.org
Components: Blink>Editing Tools>Test>FindIt>CorrectResult
Labels: -Type-Bug findit-for-crash Te-Logged Type-Bug-Regression
Owner: sigbjo...@opera.com
Status: Assigned (was: Available)
Suspected CLs	No CL in the regression range changes the crashed files. The result is the blame information.

Author: sigbjornf
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/3e5bcb132b1af136ba6faa1c152dcba2c298549f
Time: Mon May 23 14:28:16 2016
The CL last changed line 75 of file Member.h, which is stack frame 0.

Author: darin
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/33030df92939cbc4e3097d21ba3995b64d723093
Time: Mon Mar 20 02:57:59 2006
The CL last changed line 200 of file Node.h, which is stack frame 1.

Author: adam.treat@samsung.com
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/ee809a85a24df299c26e2f34346a4c058e4ebf8d
Time: Fri Jan 10 18:47:04 2014
The CL last changed line 293 of file NodeTraversal.h, which is stack frame 2.

Author: keishi
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/54c2f4c9998fc76d13ba11e75cffbfa0f9763c35
Time: Mon Apr 11 04:04:55 2016
The CL last changed line 1117 of file CompositeEditCommand.cpp, which is stack frame 3.

Author: yosin
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/87a847835586d42e3a097a8659b071883b4c83a1
Time: Wed Feb 10 07:20:01 2016
The CL last changed line 1209 of file CompositeEditCommand.cpp, which is stack frame 4.

Author: keishi
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/54c2f4c9998fc76d13ba11e75cffbfa0f9763c35
Time: Mon Apr 11 04:04:55 2016
The CL last changed line 103 of file FormatBlockCommand.cpp, which is stack frame 5.

Author: tkent
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/7f6bd2b6a8e6e4858afd1f1b23d768030a01af69
Time: Wed Feb 10 02:54:06 2016
The CL last changed line 145 of file ApplyBlockElementCommand.cpp, which is stack frame 6.

Suspected Project: chromium-blink
Suspected Component: Blink>DOM

From the above Find It result, suspecting: https://codereview.chromium.org/1999363002 from stack frame 0.

sigbjornf@: Could you please take a look at this.

Thank you!

Comment 2 by yosin@chromium.org, Jul 7 2016

Labels: -Pri-1 Pri-2
Owner: ----
Status: Available (was: Assigned)
Summary: FormatBlock command crashes with unusual HTML (was: Crash in blink::CompositeEditCommand::cloneParagraphUnderNewElement)
Hit DCHECK(a.isNotNull()) in comparePositions(a, b)

It seems endOfNextParagrahSplittingTextNodesIfNeeded() makes |start| to null,
since Text node is out of BODY element.

DOM tree at assertion:
endOfParagraphToMove.showTreeForThis()
#document	0000025CCCA024A0
	HTML	0000025CCCA02FD0 (editable)
		BODY	0000025CCCA032E8 CLASS="CLASS9 CLASS7" (editable)
			#text	0000025CCCA03350 "\n"
			INS	0000025CCCA033A0 (editable)
				#text	0000025CCCA03408 "\n"
				svg	0000025CCCA03458 (editable)
					#text	0000025CCCA035B0 "\n"
			#text	0000025CCCA03600 ""
			HEAD	0000025CCCA03038 (editable)
				SCRIPT	0000025CCCA030A0 (editable)
					#text	0000025CCCA03118 "...script..."
				#text	0000025CCCA03168 "\n"
				STYLE	0000025CCCA031B8 (editable)
					#text	0000025CCCA03248 "\n.CLASS7{-webkit-hyphens:initial;display:inline;"
				#text	0000025CCCA03298 "\n"
			DL	0000025CCCA036F0 (editable)
			#text	0000025CCCA036A0 "\n"
*		#text	0000025CCCA03650 "i4****[S[[[[[[2:3}}}}}}}}}}}}"J'"




Project Member

Comment 3 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 4 by ClusterFuzz, Dec 22 2016

Status: WontFix (was: Available)
ClusterFuzz testcase 6634147296837632 is flaky and no longer reproduces, so closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.

Sign in to add a comment