New issue
Advanced search Search tips

Issue 599849 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Heap-use-after-free in blink::LayoutBoxModelObject::invalidateStickyConstraints

Project Member Reported by ClusterFuzz, Apr 1 2016

Issue description

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

Fuzzer: attekett_dom_fuzzer
Job Type: linux_tsan_chrome_mp
Platform Id: linux

Crash Type: Heap-use-after-free READ 8
Crash Address: 0x7d340000b000
Crash State:
  blink::LayoutBoxModelObject::invalidateStickyConstraints
  blink::LayoutBlock::updateAfterLayout
  blink::LayoutFlexibleBox::layoutBlock
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=384213:384232

Minimized Testcase (0.20 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv962gi8VyabVMhXiClu5gqygSWmTAPSi_Kgbp1yh4Zfa_44h9zgGglk5vbdlwiHJDTukJ5mme624YLg7N7eEasf2LDbmIy4xTWQ2nk-dQHfK6LKGN0zlSwTiLVBeRvpa-e7Bc1so9rxhdnnds1xQ7f1DM8D5eQ
<style>
a:hover {
}
.face {
    overflow: hidden;
    top: 155px;
</style>
    <div class="face left">
    <video>  
<script> 
setTimeout(function(){
document.styleSheets[0].disabled=true;;
})
</script>


Filer: mmoroz

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: mmoroz@chromium.org
Owner: dsinclair@chromium.org
dsinclair@, could you please take a look or suggest another owner?
Components: Blink>Layout
Project Member

Comment 3 by ClusterFuzz, Apr 1 2016

Labels: Pri-1
Status: Assigned (was: Available)
Project Member

Comment 4 by ClusterFuzz, Apr 2 2016

ClusterFuzz has detected this issue as fixed in range 384282:384380.

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

Fuzzer: attekett_dom_fuzzer
Job Type: linux_tsan_chrome_mp
Platform Id: linux

Crash Type: Heap-use-after-free READ 8
Crash Address: 0x7d340000b000
Crash State:
  blink::LayoutBoxModelObject::invalidateStickyConstraints
  blink::LayoutBlock::updateAfterLayout
  blink::LayoutFlexibleBox::layoutBlock
  
Recommended Security Severity: High

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=384213:384232
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_tsan_chrome_mp&range=384282:384380

Minimized Testcase (0.20 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv962gi8VyabVMhXiClu5gqygSWmTAPSi_Kgbp1yh4Zfa_44h9zgGglk5vbdlwiHJDTukJ5mme624YLg7N7eEasf2LDbmIy4xTWQ2nk-dQHfK6LKGN0zlSwTiLVBeRvpa-e7Bc1so9rxhdnnds1xQ7f1DM8D5eQ
<style>
a:hover {
}
.face {
    overflow: hidden;
    top: 155px;
</style>
    <div class="face left">
    <video>  
<script> 
setTimeout(function(){
document.styleSheets[0].disabled=true;;
})
</script>


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 sheriffbot@chromium.org, Apr 2 2016

Labels: M-51
Project Member

Comment 6 by ClusterFuzz, Apr 2 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
Status: Fixed (was: Assigned)
Closing as per clusterfuzz in #4.
Project Member

Comment 8 by ClusterFuzz, Apr 5 2016

Labels: -Restrict-View-SecurityTeam Merge-Triage Restrict-View-SecurityNotify
Adding Merge-Triage label for tracking purposes.

Once your fix had sufficient bake time (on canary, dev as appropriate), please nominate your fix for merge by adding the Merge-Requested label.

When your merge is approved by the release manager, please start merging with higher milestone label first. Make sure to re-request merge for every milestone in the label list. You can get branch information on omahaproxy.appspot.com.

Your fix is very close to the branch point. After the branch happens, please make sure to check if your fix is in.

- Your friendly ClusterFuzz
Labels: -Merge-Triage merge-na
Labels: -reward-topanel reward-unpaid reward-3500
Atte - another $3,500 for you here ($3k for the report, $500 for the fuzzer).

Comment 11 Deleted

Comment 12 Deleted

Labels: -rewardto-attekettatgmail.com reward_to-attekett_at_gmail.com
Labels: -reward-unpaid reward-inprocess
Project Member

Comment 15 by sheriffbot@chromium.org, Jul 12 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 16 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 17 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