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

Issue 617882 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Crash in v8::internal::PointerUpdateJobTraits<

Project Member Reported by ClusterFuzz, Jun 7 2016

Issue description

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x7fb7f253e138
Crash State:
  v8::internal::PointerUpdateJobTraits<
  int v8::internal::SlotSet::Iterate<v8::internal::PointerUpdateJobTraits<
  v8::internal::PointerUpdateJobTraits<
  
Recommended Security Severity: Medium

Regressed: V8: r36762:36763

Minimized Testcase (0.41 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv96wLe_7jEpXSK_ZrnYZCT4X0MLlbfLBuAhe9PFb-UEohV6ZgphMEPdNSGoFeJS616PVfUTO4WyqKPdieXBskyZyVYDzeKRKnria1oQKqwLiAeSln2--raORADS_ZfLo-8JwYKrkPQrV_yRxgnfFTG1Xa4Yshg
var __v_33 = {};
__v_4 = 70000;
function __f_18() {
  if ((__v_7 % 50) != 0) {
  } else {
    return __v_33 + 0.5;
  }
}
function __f_17(a) {
  for (var __v_7= 0; __v_7 < __v_4; ++__v_7 ) {
    a[__v_7] = __f_18();
  }
}
for (var __v_7= 0; __v_7 < __v_4; __v_7 += 500 ) {
}
__v_9 = new Array();
__f_17(__v_9);
__v_9.length = 100;
Debug = debug.Debug
function __f_26() {
      }
__v_29 = "(function() {\
         })()";


Filer: mstarzinger

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: u...@chromium.org
Owner: hpayer@chromium.org
Status: Assigned (was: Available)
Regression range points to d61a5c376ba51145dc4684e39d5d3a9ce75bcfa6.
Project Member

Comment 2 by sheriffbot@chromium.org, Jun 7 2016

Labels: Pri-1
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 7 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/v8/v8.git/+/67af060318aef99e1a8d8c3d9eab771a930e5c3e

commit 67af060318aef99e1a8d8c3d9eab771a930e5c3e
Author: machenbach <machenbach@chromium.org>
Date: Tue Jun 07 18:28:05 2016

Revert of [heap] Clear out of live range remembered set slots in large objects. (patchset #2 id:20001 of https://codereview.chromium.org/2043713006/ )

Reason for revert:
Fails arm sim:
https://build.chromium.org/p/client.v8.ports/builders/V8%20Linux%20-%20arm%20-%20sim/builds/1012

Original issue's description:
> [heap] Clear out of live range remembered set slots in large objects.
>
> BUG= chromium:617882 
> LOG=n
>
> Committed: https://crrev.com/38ad63ff15d7e379423be4c57ae94ae2c9ffb4af
> Cr-Commit-Position: refs/heads/master@{#36795}

TBR=ulan@chromium.org,hpayer@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= chromium:617882 

Review-Url: https://codereview.chromium.org/2042403002
Cr-Commit-Position: refs/heads/master@{#36799}

[modify] https://crrev.com/67af060318aef99e1a8d8c3d9eab771a930e5c3e/src/heap/spaces.cc
[modify] https://crrev.com/67af060318aef99e1a8d8c3d9eab771a930e5c3e/src/heap/spaces.h
[delete] https://crrev.com/839f3fd406426a221d74eb7a33a72794c3c7a548/test/mjsunit/regress/regress-617882.js

Project Member

Comment 5 by ClusterFuzz, Jun 9 2016

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_dbg
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x7fbf70d488e0
Crash State:
  v8::internal::PointerUpdateJobTraits<
  int v8::internal::SlotSet::Iterate<v8::internal::PointerUpdateJobTraits<
  v8::internal::PointerUpdateJobTraits<
  
Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_d8_ignition_dbg&range=36798:36799

Minimized Testcase (0.28 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv947Ivgoypm6xIBzd1akFZYJ5bJrMOTqVFA2DEw8XfMjVXdvIgtVOn8qHb_IRQBFeH_d5BIQ5dp7PtC1r9vUwFL_GwEoFcA6KWOvqBgI0BSPjPCNkk4KpxsWImPxBclBhheFGtC5J0S7jQKTplaKC8pv8id71g
__v_0 = 70000;
function __f_14(__v_3) {
  if ((__v_3 % 50) != 0) {
    return __v_3 + 0.5;
  }
}
function __f_13(a) {
  for (var __v_3= 0; __v_3 < __v_0; ++__v_3 ) {
    a[__v_3] = __f_14(__v_3);
  }
}
function __f_2() {
}
__v_5 = new Array();
__f_13(__v_5);
__v_5.length = 25;
gc();


Filer: mstarzinger

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
Project Member

Comment 6 by ClusterFuzz, Jun 9 2016

Labels: Security_Impact-Stable
Project Member

Comment 7 by ClusterFuzz, Jun 10 2016

ClusterFuzz has detected this issue as fixed in range 36806:36807.

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_dbg
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x7fbf70d488e0
Crash State:
  v8::internal::PointerUpdateJobTraits<
  int v8::internal::SlotSet::Iterate<v8::internal::PointerUpdateJobTraits<
  v8::internal::PointerUpdateJobTraits<
  
Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_d8_ignition_dbg&range=36798:36799
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_d8_ignition_dbg&range=36806:36807

Minimized Testcase (0.28 Kb):
Download: https://cluster-fuzz.appspot.com/download/AMIfv947Ivgoypm6xIBzd1akFZYJ5bJrMOTqVFA2DEw8XfMjVXdvIgtVOn8qHb_IRQBFeH_d5BIQ5dp7PtC1r9vUwFL_GwEoFcA6KWOvqBgI0BSPjPCNkk4KpxsWImPxBclBhheFGtC5J0S7jQKTplaKC8pv8id71g
__v_0 = 70000;
function __f_14(__v_3) {
  if ((__v_3 % 50) != 0) {
    return __v_3 + 0.5;
  }
}
function __f_13(a) {
  for (var __v_3= 0; __v_3 < __v_0; ++__v_3 ) {
    a[__v_3] = __f_14(__v_3);
  }
}
function __f_2() {
}
__v_5 = new Array();
__f_13(__v_5);
__v_5.length = 25;
gc();


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: M-52
Project Member

Comment 9 by ClusterFuzz, Jun 14 2016

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

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

Comment 10 by ClusterFuzz, Jun 14 2016

Labels: Merge-Triage M-51
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-Request-XX label, where XX is the Chrome milestone.

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 friendly ClusterFuzz
Project Member

Comment 11 by sheriffbot@chromium.org, Jun 14 2016

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Labels: -ClusterFuzz-Verified
Status: Assigned (was: Verified)
Sorry for the incorrect ClusterFuzz update, please ignore it. Reopening bug.
Project Member

Comment 13 by ClusterFuzz, Jun 15 2016

Status: Fixed (was: Assigned)
Please mark security bugs as fixed as soon as the fix lands, and before requesting merges.

- Your friendly ClusterFuzz
Status: Assigned (was: Fixed)
I am confused, why is it again marked as fixed?
Labels: -Merge-Triage
It was because of the Merge-Triage label, which is only expected to be applied after an issue is fixed.
Project Member

Comment 16 by sheriffbot@chromium.org, Jun 21 2016

hpayer: Uh oh! This issue still open and hasn't been updated in the last 14 days. This is a serious vulnerability, and we want to ensure that there's progress. Could you please leave an update with the current status and any potential blockers?

If you're not the right owner for this issue, could you please remove yourself as soon as possible or help us find the right one?

If the issue is fixed or you can't reproduce it, please close the bug. If you've started working on a fix, please set the status to Started.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Assigned)
The involved CLs were reverted shortly after landing. Nothing to do here.
Labels: -Security_Impact-Stable Security_Impact-None
Security_Impact-None per #17
Project Member

Comment 19 by sheriffbot@chromium.org, Oct 5 2016

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

Sign in to add a comment