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

Issue 599631 link

Starred by 0 users

Issue metadata

Status: WontFix
Owner:
NOT IN USE
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Security



Sign in to add a comment

Crash in blink::PaintLayerReflectionInfo::updateAfterStyleChange

Project Member Reported by ClusterFuzz, Mar 31 2016

Issue description

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

Fuzzer: mbarbella_js_mutation_layout
Job Type: windows_asan_content_shell
Platform Id: windows

Crash Type: UNKNOWN READ
Crash Address: 0xffffffff
Crash State:
  blink::PaintLayerReflectionInfo::updateAfterStyleChange
  blink::PaintLayer::styleChanged
  blink::LayoutBoxModelObject::styleDidChange
  
Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_asan_content_shell&range=379564:379959

Minimized Testcase (0.07 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97nSGetEC-stEO39M_gFP_zWGJ71xG-M2cUWUGkfG7WRDPyuJVtA5DkcpoBmbm0pXLzzZKuO516yq0XmP0wsxZQzWU1hC5S1V1Xc0W0F2Aw2EgNdTMX3gnlGLQ3lsciogugiwS3Keq1bE34oAn2jflOIcHAhA
<style>
   div {
        -webkit-box-reflect: below;
  </style>
  <div>


Filer: inferno

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Owner: danakj@chromium.org
Status: Assigned (was: Available)
Author: danakj
Component: chromium
Changelist: https://chromium.googlesource.com/chromium/src//+/55d0a18194e3abe3ef52f48f021fa677c4803153
Time: Tue Mar 08 21:33:26 2016
Files FrameView.cpp, Element.cpp are changed in this cl (and is part of stack frame #19, "blink::FrameView::updateLifecyclePhasesInternal"; frame #20, "blink::FrameView::updateAllLifecyclePhases")
Minimum distance from crash line to modified line: 2. (file: Element.cpp, crashed on: 1548, modified: 1550).

Comment 2 by danakj@chromium.org, Mar 31 2016

Owner: ymalik@chromium.org
The lines I changed had to do with scrolling. I don't think the enum name matters. But there is some scroll-related change in that list. https://chromium.googlesource.com/chromium/src/+/d0eaee288867a6e00e4a9c3fa65d28e4de9e922a
Project Member

Comment 3 by ClusterFuzz, Mar 31 2016

Labels: Pri-1
Project Member

Comment 4 by sheriffbot@chromium.org, Apr 1 2016

Labels: M-51
Cc: bokan@chromium.org
Project Member

Comment 6 by ClusterFuzz, Apr 1 2016

Labels: ReleaseBlock-Beta
This medium+ severity security issue is a regression on trunk.

Please fix this asap. If you are unable to look into this soon, please revert your change.

- Your friendly ClusterFuzz
Cc: dtapu...@chromium.org
Cc: ymalik@chromium.org
Owner: e...@chromium.org
I don't think my CL would have caused this crash. My CL has to do with scrolling, more precisely taking over animations from CC. AFAICT, the minimized test case doesn't have any scrolling and the stacktrace suggests a layout issue.

I tried to repro this locally, but wasn't successful.

@eae, can you please re-assign this as necessary?
Components: Blink>Layout

Comment 10 by e...@chromium.org, Apr 5 2016

Cc: e...@chromium.org
Owner: r...@opera.com
r379762 (https://codereview.chromium.org/1769903002) is the only change in that range that looks related. Would you mind taking a look rune?

Project Member

Comment 11 by ClusterFuzz, Apr 5 2016

ClusterFuzz has detected this testcase as flaky and is unable to reproduce it in the original crash revision. Skipping fixed testing check and marking it as potentially fixed.

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

Fuzzer: mbarbella_js_mutation_layout
Job Type: windows_asan_content_shell
Platform Id: windows

Crash Type: UNKNOWN READ
Crash Address: 0xffffffff
Crash State:
  blink::PaintLayerReflectionInfo::updateAfterStyleChange
  blink::PaintLayer::styleChanged
  blink::LayoutBoxModelObject::styleDidChange
  
Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_asan_content_shell&range=379564:379959

Minimized Testcase (0.07 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv97nSGetEC-stEO39M_gFP_zWGJ71xG-M2cUWUGkfG7WRDPyuJVtA5DkcpoBmbm0pXLzzZKuO516yq0XmP0wsxZQzWU1hC5S1V1Xc0W0F2Aw2EgNdTMX3gnlGLQ3lsciogugiwS3Keq1bE34oAn2jflOIcHAhA
<style>
   div {
        -webkit-box-reflect: below;
  </style>
  <div>


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.

Comment 12 by e...@chromium.org, Apr 5 2016

Status: WontFix (was: Assigned)
Never mind.
Project Member

Comment 13 by sheriffbot@chromium.org, Jul 13 2016

Labels: -Restrict-View-SecurityTeam
This bug has been closed for more than 14 weeks. Removing security view restrictions.

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

Comment 14 by sheriffbot@chromium.org, Oct 1 2016

This bug has been closed for more than 14 weeks. Removing security view restrictions.

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

Comment 15 by sheriffbot@chromium.org, Oct 2 2016

This bug has been closed for more than 14 weeks. Removing security view restrictions.

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

Sign in to add a comment