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

Issue 600735 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Email to this user bounced
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Bug-Security



Sign in to add a comment

Heap-use-after-free in blink::LayoutObject::isAnonymousBlock

Project Member Reported by ClusterFuzz, Apr 5 2016

Issue description

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

Fuzzer: bj_broddelwerk
Job Type: windows_syzyasan_chrome
Platform Id: windows

Crash Type: Heap-use-after-free READ 4
Crash Address: 0x26c7ef97
Crash State:
  blink::LayoutObject::isAnonymousBlock
  blink::LayoutListItem::updateMarkerLocation
  blink::LayoutListItem::subtreeDidChange
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_chrome&range=384870:384903

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv95rxtOKfp0y9z2e7rCKS1dRs8iTx-o0mg9KdJ88CTT8m6sNMGQDw2FI9RrVblYr4nNMAPuHISFGea5KNpb97Cbc-C9P2PH6CrYRj3WDWT1SxzdXIGfUVGDJdDgwy6IruiOvDyZZzeMuumVnbB4dJO0eIkHWfyLw66xTxIIdG5ZJl11w3Iw


Filer: mmoroz

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: mmoroz@chromium.org mbarbe...@chromium.org
Components: Blink>Layout
Labels: M-50 Pri-1
Owner: dsinclair@chromium.org
dsinclair@, could you please help to find an owner for that?
Owner: e...@chromium.org
eae@ to triage for layout-dev

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

Cc: haraken@chromium.org
Owner: sigbj...@opera.com
r384897 is the only change in the regression range that stands out and it does change lifecycle management slightly.

https://codereview.chromium.org/1858583002/
I don't think https://codereview.chromium.org/1858583002/ is related since it just removed dead code.

Project Member

Comment 5 by ClusterFuzz, Apr 6 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=5612673827340288

Fuzzer: bj_broddelwerk
Job Type: windows_syzyasan_chrome
Platform Id: windows

Crash Type: Heap-use-after-free READ 4
Crash Address: 0x26c7ef97
Crash State:
  blink::LayoutObject::isAnonymousBlock
  blink::LayoutListItem::updateMarkerLocation
  blink::LayoutListItem::subtreeDidChange
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_chrome&range=384870:384903

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv95rxtOKfp0y9z2e7rCKS1dRs8iTx-o0mg9KdJ88CTT8m6sNMGQDw2FI9RrVblYr4nNMAPuHISFGea5KNpb97Cbc-C9P2PH6CrYRj3WDWT1SxzdXIGfUVGDJdDgwy6IruiOvDyZZzeMuumVnbB4dJO0eIkHWfyLw66xTxIIdG5ZJl11w3Iw


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 6 by ClusterFuzz, Apr 6 2016

Status: Assigned (was: Available)
Project Member

Comment 7 by sheriffbot@chromium.org, Apr 6 2016

Labels: -Security_Impact-Head Security_Impact-Beta
Project Member

Comment 8 by ClusterFuzz, Apr 6 2016

Labels: ReleaseBlock-Stable
M50 Stable is launching very soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged ASAP. All changes MUST be merged into the release branch by 5pm on Apr-8 to make into the desktop Stable final build cut. Thanks!
Cc: timwillis@chromium.org
+ timwillis@ [Security TPM]
May be I was wrong when added M-50 label, feel free to change it to M-51 if needed.
timwillis@, could you please confirm whether this should be M-50 OR M-51 Stable blocker. Thank you.
Project Member

Comment 13 by ClusterFuzz, Apr 7 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=5612673827340288

Fuzzer: bj_broddelwerk
Job Type: windows_syzyasan_chrome
Platform Id: windows

Crash Type: Heap-use-after-free READ 4
Crash Address: 0x26c7ef97
Crash State:
  blink::LayoutObject::isAnonymousBlock
  blink::LayoutListItem::updateMarkerLocation
  blink::LayoutListItem::subtreeDidChange
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_syzyasan_chrome&range=384870:384903

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv95rxtOKfp0y9z2e7rCKS1dRs8iTx-o0mg9KdJ88CTT8m6sNMGQDw2FI9RrVblYr4nNMAPuHISFGea5KNpb97Cbc-C9P2PH6CrYRj3WDWT1SxzdXIGfUVGDJdDgwy6IruiOvDyZZzeMuumVnbB4dJO0eIkHWfyLw66xTxIIdG5ZJl11w3Iw


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.
Labels: -Security_Impact-Beta -ReleaseBlock-Stable Security_Impact-Head merge-na
Status: Fixed (was: Assigned)
redid testing - seems like crash was flaky and can't reproduce. Marking as Fixed with no merge required
Project Member

Comment 15 by ClusterFuzz, Apr 8 2016

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Cool, thanks Tim!
Project Member

Comment 17 by sheriffbot@chromium.org, Jul 15 2016

Labels: -Restrict-View-SecurityNotify
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 18 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 19 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