New issue
Advanced search Search tips

Issue 792727 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Closed: Dec 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Null-dereference WRITE in blink::ObjectPaintPropertyTreeBuilder::UpdateForSelf

Project Member Reported by ClusterFuzz, Dec 7 2017

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=4738715716157440

Fuzzer: miaubiz_svg_fuzzer
Job Type: mac_asan_chrome
Platform Id: mac

Crash Type: Null-dereference WRITE
Crash Address: 0x00000000000f
Crash State:
  blink::ObjectPaintPropertyTreeBuilder::UpdateForSelf
  blink::PrePaintTreeWalk::WalkInternal
  blink::PrePaintTreeWalk::Walk
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=522150:522195

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4738715716157440

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by ClusterFuzz, Dec 7 2017

Labels: OS-Windows OS-Linux
Project Member

Comment 2 by ClusterFuzz, Dec 7 2017

Labels: Test-Predator-Auto-Owner
Owner: wangxianzhu@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/chromium/src/+/46367c60003ed45d9fc3f2f448ca9482f8b89ffd ([PE] Update reference filters on box resize).

If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
Project Member

Comment 3 by ClusterFuzz, Dec 7 2017

Components: Blink>Paint
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Cc: wangxianzhu@chromium.org
 Issue 792901  has been merged into this issue.
Project Member

Comment 5 by bugdroid1@chromium.org, Dec 8 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bf8135d0bce64496b872062a07b427ddff825ec5

commit bf8135d0bce64496b872062a07b427ddff825ec5
Author: Xianzhu Wang <wangxianzhu@chromium.org>
Date: Fri Dec 08 01:05:27 2017

[PE] Avoid null pointer for objects with reflection but no effect

Reflection only has effect if we create layer for the object. Some
objects (e.g. SVG text) having reflection style but doesn't create
layers. Should check for the condition.

Bug:  792727 , 777259 
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I09528df085002f8d30e1f20fd07a7e7b3f2d3cf1
Reviewed-on: https://chromium-review.googlesource.com/814894
Commit-Queue: Chris Harrelson <chrishtr@chromium.org>
Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#522661}
[modify] https://crrev.com/bf8135d0bce64496b872062a07b427ddff825ec5/third_party/WebKit/Source/core/layout/LayoutBox.cpp
[modify] https://crrev.com/bf8135d0bce64496b872062a07b427ddff825ec5/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilder.cpp
[modify] https://crrev.com/bf8135d0bce64496b872062a07b427ddff825ec5/third_party/WebKit/Source/core/paint/PaintPropertyTreeBuilderTest.cpp

Project Member

Comment 6 by ClusterFuzz, Dec 8 2017

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

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

Comment 7 by ClusterFuzz, Dec 8 2017

ClusterFuzz has detected this issue as fixed in range 522600:522669.

Detailed report: https://clusterfuzz.com/testcase?key=4738715716157440

Fuzzer: miaubiz_svg_fuzzer
Job Type: mac_asan_chrome
Platform Id: mac

Crash Type: Null-dereference WRITE
Crash Address: 0x00000000000f
Crash State:
  blink::ObjectPaintPropertyTreeBuilder::UpdateForSelf
  blink::PrePaintTreeWalk::WalkInternal
  blink::PrePaintTreeWalk::Walk
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=522150:522195
Fixed: https://clusterfuzz.com/revisions?job=mac_asan_chrome&range=522600:522669

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4738715716157440

See https://github.com/google/clusterfuzz-tools for more information.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.

Sign in to add a comment