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

Issue 692462 link

Starred by 1 user

Issue metadata

Status: Verified
Owner: ----
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

ApplyStyle command crashes with unusual HTML

Project Member Reported by ClusterFuzz, Feb 15 2017

Issue description

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_lsan_chrome_mp
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000010
Crash State:
  getFlag
  isTextNode
  blink::ApplyStyleCommand::applyRelativeFontStyleChange
  
Sanitizer: address (ASAN)

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=428077:428348

Reproducer Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97XCtg_ZegQBMpu8BG7BMqEuqLTGYZQzrdvihN7B12z3TWcoJn-8_LtMeezVMe_6I63lxbDBC8tGXBm-A7X-cs4F-hwYU5QF1T-aI3TiJ1ZHQSL6gY0RP4fA7QkfG-X9h0GxPUCM8vx6HngWxYe4InnuSGH5g?testcase_id=5895503083732992


Additional requirements: Requires Gestures

Issue filed automatically.

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: msrchandra@chromium.org
Components: Blink>Editing
Labels: M-57 Test-Predator-Correct-CLs
Owner: yosin@chromium.org
Status: Assigned (was: Untriaged)
Assigning to the concern owner from Predator results --
The result is a list of CLs that change the crashed files. 

Author: yosin
Project: chromium
Changelist: https://chromium.googlesource.com/chromium/src/+/e3d2a5b25ef09f4c8458db05080e604efae9c1aa
Time: Fri Oct 28 07:19:30 2016
Lines 300 of file CompositeEditCommand.cpp which potentially caused crash are changed in this cl (frame #6, "blink::CompositeEditCommand::applyCommandToComposite"). 

Files ApplyStyleCommand.cpp, InsertTextCommand.cpp are changed in this cl (and is part of stack frame #2, "blink::ApplyStyleCommand::applyRelativeFontStyleChange"; frame #3, "blink::ApplyStyleCommand::doApply")
Minimum distance from crash line to modified line: 0. (file: CompositeEditCommand.cpp, crashed on: 300, modified: 300).

@yosin -- Could you please look into the issue, kindly re-assign if this is not related to your changes.
Thank You.

Comment 2 by yosin@chromium.org, Feb 18 2017

Components: -Blink>Editing Blink>Editing>Command
Owner: ----
Status: Available (was: Assigned)
Summary: ApplyStyle command crashes with unusual HTML (was: Crash in getFlag)
Lower to Pri-2, since real world usage of ApplyStyle command is low.

Comment 3 by yosin@chromium.org, Feb 18 2017

Labels: -Pri-1 Pri-2
Project Member

Comment 4 by ClusterFuzz, Feb 24 2017

ClusterFuzz has detected this issue as fixed in range 452175:452490.

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

Fuzzer: inferno_layout_test_unmodified
Job Type: linux_lsan_chrome_mp
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000010
Crash State:
  getFlag
  isTextNode
  blink::ApplyStyleCommand::applyRelativeFontStyleChange
  
Sanitizer: address (ASAN)

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=428077:428348
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_lsan_chrome_mp&range=452175:452490

Reproducer Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97XCtg_ZegQBMpu8BG7BMqEuqLTGYZQzrdvihN7B12z3TWcoJn-8_LtMeezVMe_6I63lxbDBC8tGXBm-A7X-cs4F-hwYU5QF1T-aI3TiJ1ZHQSL6gY0RP4fA7QkfG-X9h0GxPUCM8vx6HngWxYe4InnuSGH5g?testcase_id=5895503083732992


Additional requirements: Requires Gestures

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.
Project Member

Comment 5 by ClusterFuzz, Feb 24 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Available)
ClusterFuzz testcase 5895503083732992 is verified as fixed, so closing issue.

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

Sign in to add a comment