New issue
Advanced search Search tips

Issue 904275 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 904176
Owner:
Closed: Nov 12
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Unreachable code in ast-traversal-visitor.h

Project Member Reported by ClusterFuzz, Nov 12

Issue description

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_d8_dbg
Platform Id: linux

Crash Type: Unreachable code
Crash Address: 
Crash State:
  ast-traversal-visitor.h
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=57318:57319

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Nov 12

Labels: Test-Predator-Auto-Owner
Owner: verwa...@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/v8/v8/+/9d27ec3aacf76b8131c323dd57968209e6f6ff4c ([parser] Introduce FailureExpression to return instead of nullptr).

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.
Mergedinto: 904176
Status: Duplicate (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 12

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

commit cdae5af8967d0efa73340e4df9831e194dfe74b9
Author: Toon Verwaest <verwaest@chromium.org>
Date: Mon Nov 12 09:16:50 2018

[parser] Don't reindex function literals if there's a parser error

Bug:  chromium:904275 
Change-Id: I82712f70954246c16846f44bb9e8dc6e60e73d83
Reviewed-on: https://chromium-review.googlesource.com/c/1329686
Reviewed-by: Igor Sheludko <ishell@chromium.org>
Commit-Queue: Toon Verwaest <verwaest@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57421}
[modify] https://crrev.com/cdae5af8967d0efa73340e4df9831e194dfe74b9/src/parsing/parser.cc
[add] https://crrev.com/cdae5af8967d0efa73340e4df9831e194dfe74b9/test/mjsunit/regress/regress-904275.js

Project Member

Comment 4 by ClusterFuzz, Nov 13

ClusterFuzz has detected this issue as fixed in range 57420:57421.

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_d8_dbg
Platform Id: linux

Crash Type: Unreachable code
Crash Address: 
Crash State:
  ast-traversal-visitor.h
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=57318:57319
Fixed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=57420:57421

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

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.

Sign in to add a comment