New issue
Advanced search Search tips

Issue 724608 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

CHECK failure: !map->is_deprecated() in compilation-dependencies.cc

Project Member Reported by ClusterFuzz, May 19 2017

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=4852572602236928

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_v8_arm_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !map->is_deprecated() in compilation-dependencies.cc
  v8::internal::CompilationDependencies::AssumeMapNotDeprecated
  v8::internal::compiler::AccessInfoFactory::LookupTransition
  
Sanitizer: address (ASAN)

Regressed: V8: 43683:43684

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4852572602236928


Issue filed automatically.

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

Comment 1 by och...@chromium.org, May 24 2017

Labels: Type-Bug-Security
Applying security view restrictions to all v8 CHECK/DCHECK failures.

(CHECKs aren't security, but we have no way to distinguish these right now).

Comment 2 by aarya@google.com, May 25 2017

Owner: ishell@chromium.org

Comment 3 by aarya@google.com, May 25 2017

Cc: mstarzinger@chromium.org

Comment 4 by ishell@chromium.org, May 26 2017

Status: Assigned (was: Untriaged)

Comment 5 by ishell@chromium.org, May 26 2017

Cc: ishell@chromium.org jarin@chromium.org
Owner: bmeu...@chromium.org
We are trying to search transitions in a deprecated receiver map and are surprised that the transition map is also deprecated. We should probably try to update the receiver map first.
Labels: Security_Impact-Beta
Cc: mbarbe...@chromium.org
I'm not sure of the severity on this one. Should this be marked as a security bug? If so, what's the impact of it.
Project Member

Comment 8 by bugdroid1@chromium.org, Jun 6 2017

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

commit 468446d5fc24148086097aac0ed166c02da56416
Author: bmeurer <bmeurer@chromium.org>
Date: Tue Jun 06 12:10:40 2017

[turbofan] Try to update deprecated maps first.

When optimizing stores to data properties in literals, we need to first
migrate deprecated maps before we lookup the property access infos for
those.

BUG= chromium:724608 
R=jarin@chromium.org

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

[modify] https://crrev.com/468446d5fc24148086097aac0ed166c02da56416/src/compiler/js-native-context-specialization.cc
[add] https://crrev.com/468446d5fc24148086097aac0ed166c02da56416/test/mjsunit/regress/regress-crbug-724608.js

Status: Fixed (was: Assigned)
Project Member

Comment 10 by sheriffbot@chromium.org, Jun 6 2017

Labels: Restrict-View-SecurityNotify
Project Member

Comment 11 by ClusterFuzz, Jun 7 2017

ClusterFuzz has detected this issue as fixed in range 45726:45727.

Detailed report: https://clusterfuzz.com/testcase?key=4852572602236928

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_v8_arm_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !map->is_deprecated() in compilation-dependencies.cc
  v8::internal::CompilationDependencies::AssumeMapNotDeprecated
  v8::internal::compiler::AccessInfoFactory::LookupTransition
  
Sanitizer: address (ASAN)

Regressed: V8: 43683:43684
Fixed: V8: 45726:45727

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4852572602236928


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 12 by sheriffbot@chromium.org, Sep 12 2017

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