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

Issue 649321 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug

Blocking:
issue v8:5634



Sign in to add a comment

Cyclic object state detected by escape analysis in escape-analysis-reducer.cc

Project Member Reported by ClusterFuzz, Sep 22 2016

Issue description

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  Cyclic object state detected by escape analysis in escape-analysis-reducer.cc
  

Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv96R6kknQYlYTH1l9u88xBlUEWBSfBdmIAGHz5c2b-AMWfvwYZg8ca3JNwDmQbeOmUYmBOhp81nitjVjr_m6ThwbKYJf7xdyaNGpO3D8eYkTufjJ5CzpwCwH7LrrFfVcJb3GBNCSOmQoo3pEXWRnF1vsDFkReQ?testcase_id=5846028312117248


Issue manually filed by: mstarzinger

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: -mstarzinger@chromium.org ishell@chromium.org
Components: -Blink>JavaScript Blink>JavaScript>Compiler
Labels: -Pri-1 Pri-2
Owner: mstarzinger@chromium.org
Status: Assigned (was: Untriaged)
Copying simplified repro from another issue ...

function f() {
  var o1 = { a:99, val:23 };
  var o2 = { b:o1, val:42 };
  o1.a = o2;
  %DeoptimizeNow();
  return o1.a.val + o2.b.val;
}

f();
f();
%OptimizeFunctionOnNextCall(f);
f();
Cc: mstarzinger@chromium.org titzer@chromium.org jarin@chromium.org rossberg@chromium.org machenb...@chromium.org bmeu...@chromium.org
 Issue 613923  has been merged into this issue.
Project Member

Comment 4 by ClusterFuzz, Sep 27 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.
Labels: ClusterFuzz-Wrong
Status: Assigned (was: Verified)
Project Member

Comment 6 by ClusterFuzz, Oct 5 2016

ClusterFuzz has detected this issue as fixed in range 39970:39971.

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  Cyclic object state detected by escape analysis in escape-analysis-reducer.cc
  
Regressed: V8: r39613:39614
Fixed: V8: r39970:39971

Minimized Testcase (0.35 Kb): https://cluster-fuzz.appspot.com/download/AMIfv95JT-SjJ05AbgnGsjyYgpid6H_MHoqouAgUR6hMDm60x-P26FoaTzGWNyxRGW-ScrDRi_Zo8lm-cN1xVd22gZyJPmoOrwDaCA8PlNUgUQPObdKZE2wMhxvQLd0BqRUE8NL0TrQviQ1uPIyGdrSb6xEylFRndQ?testcase_id=5846028312117248

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.
 Issue 645741  has been merged into this issue.
Blocking: v8:5634
Cc: tebbi@chromium.org
Status: Fixed (was: Assigned)
Has been converted into a silent (non-crashing) bailout as of f71260b298654942f72f769d7fe281e341e2d2ef and should no longer happen on ClusterFuzz. Kudos got to Tobias. Follow-up work is tracked separately by  issue v8:5634 . Marking this one as fixed.
Very nice! Thanks Tobias! Let's turn on escape analysis after the branch cut :-)
Project Member

Comment 11 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: littledan@chromium.org
 Issue 658518  has been merged into this issue.
Labels: -ClusterFuzz-Wrong
We have made a bunch of changes on ClusterFuzz side, so resetting ClusterFuzz-Wrong label.

Sign in to add a comment