New issue
Advanced search Search tips

Issue 665587 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Crash in v8::internal::Simulator::DecodeType2

Project Member Reported by ClusterFuzz, Nov 15 2016

Issue description

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_v8_arm_dbg
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00000004
Crash State:
  v8::internal::Simulator::DecodeType2
  v8::internal::Simulator::InstructionDecode
  v8::internal::Simulator::Execute
  
Regressed: V8: r40910:40911

Minimized Testcase (0.27 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96rkeNqb8NdzBEV6Be_s6Imbgo7NajpirfrrKYLo4eW3Ja_Fs2nJUoqzMPW6AQlSCbICJo-OJg756TCfJxtx9piVJNKAp1TjqZXufQeBRrQ_Inh8YnUFfYJEBTYpZadc_ZsDurUqk8D90GZQDed-m_yKeKyVg?testcase_id=5330810145341440

Issue manually filed by: ishell

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

Comment 1 by ishell@chromium.org, Nov 15 2016

Cc: ishell@chromium.org rmcilroy@chromium.org
Labels: -OS-Linux OS-All
Owner: mstarzinger@chromium.org
Status: Assigned (was: Untriaged)
This is an Ignition-OSR issue. Also reproduces on x64.
Status: Started (was: Assigned)
For some reason TurboFan is emitting a Store[kPointerWriteBarrier] even though the value being stored is a Smi. I am investigating.
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 16 2016

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

commit 31a8ec776241d97d59e89d46cf184342bab8720f
Author: mstarzinger <mstarzinger@chromium.org>
Date: Wed Nov 16 12:52:46 2016

[turbofan] Fix bogus representation for {kCheckTaggedHole}.

The operator in question is guaranteed to produce a tagged value that is
not equal to the-hole, it however does not guarantee the value to be a
HeapObject. The correct representation hence is {kTagged}.

R=bmeurer@chromium.org
TEST=mjsunit/regress/regress-crbug-665587
BUG= chromium:665587 

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

[modify] https://crrev.com/31a8ec776241d97d59e89d46cf184342bab8720f/src/compiler/simplified-lowering.cc
[add] https://crrev.com/31a8ec776241d97d59e89d46cf184342bab8720f/test/mjsunit/regress/regress-crbug-665587.js

Status: Fixed (was: Started)
Project Member

Comment 5 by ClusterFuzz, Nov 16 2016

ClusterFuzz has detected this issue as fixed in range 41031:41032.

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_ignition_v8_arm_dbg
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x00000004
Crash State:
  v8::internal::Simulator::DecodeType2
  v8::internal::Simulator::InstructionDecode
  v8::internal::Simulator::Execute
  
Regressed: V8: r40910:40911
Fixed: V8: r41031:41032

Minimized Testcase (0.27 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96rkeNqb8NdzBEV6Be_s6Imbgo7NajpirfrrKYLo4eW3Ja_Fs2nJUoqzMPW6AQlSCbICJo-OJg756TCfJxtx9piVJNKAp1TjqZXufQeBRrQ_Inh8YnUFfYJEBTYpZadc_ZsDurUqk8D90GZQDed-m_yKeKyVg?testcase_id=5330810145341440

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 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: jarin@chromium.org
Labels: Merge-Request-56
This needs backmerging to 5.6.
Issue 692721 has been merged into this issue.
Cc: hablich@chromium.org
Re comment #7: Talking with hablich@ it seems that there is no further release of M56 planned, so merging back to 5.6 at this point doesn't make much sense anymore. Please let me know in case there are other reasons for a merge.
Labels: -Merge-Request-56 Merge-Rejected-56
Indeed.

Closing the loop.

Sign in to add a comment