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

Issue 883059 link

Starred by 2 users

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Sep 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Security



Sign in to add a comment

DCHECK failure in is_resolved() in ast.h

Project Member Reported by ClusterFuzz, Sep 11

Issue description

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  is_resolved() in ast.h
  var
  VisitVariableProxy
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55800:55801

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

Issue filed automatically.

See https://github.com/google/clusterfuzz-tools for more information.
 
Project Member

Comment 1 by ClusterFuzz, Sep 11

Components: Blink>JavaScript>Interpreter Blink>JavaScript>Language
Labels: Test-Predator-Auto-Components
Automatically applying components based on crash stacktrace and information from OWNERS files.

If this is incorrect, please apply the Test-Predator-Wrong-Components label.
Project Member

Comment 2 by ClusterFuzz, Sep 11

Labels: Test-Predator-Auto-Owner
Owner: sattlerf@google.com
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/v8/v8/+/78f8ff95680c928e3842e75c59ebd17b7ed64c5c ([preparser] Refactor VariableProxies to use ThreadedLists interface).

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label. If you aren't the correct owner for this issue, please unassign yourself as soon as possible so it can be re-triaged.
Cc: sattlerf@google.com
 Issue 883110  has been merged into this issue.
Labels: OS-Mac Pri-1
 Issue 883119  has been merged into this issue.
Labels: OS-Windows
Cc: marja@chromium.org
Adding marja
Project Member

Comment 8 by bugdroid1@chromium.org, Sep 12

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

commit d97074915211bbd915cf36a0d255ea8bc9c38540
Author: Florian Sattler <sattlerf@google.com>
Date: Wed Sep 12 15:13:29 2018

Reland "[preparser] Refactor VariableProxies to use ThreadedLists interface"

This is a reland of 78f8ff95680c928e3842e75c59ebd17b7ed64c5c

Original change's description:
> [preparser] Refactor VariableProxies to use ThreadedLists interface
>
> Bug:  v8:7926 
> Change-Id: Idfc520b67696c8a838a0ee297ea392d416dd899e
> Reviewed-on: https://chromium-review.googlesource.com/1206292
> Commit-Queue: Florian Sattler <sattlerf@google.com>
> Reviewed-by: Igor Sheludko <ishell@chromium.org>
> Reviewed-by: Marja Hölttä <marja@chromium.org>
> Reviewed-by: Camillo Bruni <cbruni@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#55801}

Bug:  v8:7926 ,  chromium:883059 
Change-Id: Icaa496be1b4df8306fe6d623e5825909d7b0c9c5
Reviewed-on: https://chromium-review.googlesource.com/1221529
Commit-Queue: Florian Sattler <sattlerf@google.com>
Reviewed-by: Camillo Bruni <cbruni@chromium.org>
Reviewed-by: Marja Hölttä <marja@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55833}
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/BUILD.gn
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/ast/ast.h
[add] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/ast/scopes-inl.h
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/ast/scopes.cc
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/ast/scopes.h
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/parsing/preparser.h
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/src/utils.h
[add] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/test/mjsunit/regress/regress-883059.js
[modify] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/test/unittests/BUILD.gn
[add] https://crrev.com/d97074915211bbd915cf36a0d255ea8bc9c38540/test/unittests/utils/threaded-list.cc

Project Member

Comment 9 by ClusterFuzz, Sep 13

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

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

Comment 10 by ClusterFuzz, Sep 13

ClusterFuzz has detected this issue as fixed in range 55818:55819.

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

Fuzzer: mbarbella_js_mutation
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  is_resolved() in ast.h
  var
  VisitVariableProxy
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55800:55801
Fixed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=55818:55819

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

See https://github.com/google/clusterfuzz-tools 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 11 by sheriffbot@chromium.org, Sep 13

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 12 by sheriffbot@chromium.org, Dec 20

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