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

Issue 619476 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

reported_errors_end_ <= i in expression-classifier.h

Project Member Reported by ClusterFuzz, Jun 13 2016

Issue description

Comment 1 by ishell@chromium.org, Jun 13 2016

Owner: nikolaos@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 13 2016

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

commit cdec5e8d26c56067d6e9bbc86dc2d8de9778bc72
Author: nikolaos <nikolaos@chromium.org>
Date: Mon Jun 13 12:33:55 2016

Remove erroneous DCHECK related to expression classifiers

It seems that I forgot to remove the DCHECK when refactoring this
function, even though the comment had it right.  It also seems that
this is hard to trigger.  The minimal example I found, after fuzzer's
bug, was:

   eval, x[eval]

R=adamk@chromium.org
BUG= chromium:619476 
LOG=N

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

[modify] https://crrev.com/cdec5e8d26c56067d6e9bbc86dc2d8de9778bc72/src/parsing/expression-classifier.h
[add] https://crrev.com/cdec5e8d26c56067d6e9bbc86dc2d8de9778bc72/test/mjsunit/regress-crbug-619476.js

Status: Fixed (was: Assigned)
Project Member

Comment 5 by ClusterFuzz, Jun 13 2016

ClusterFuzz has detected this issue as fixed in range 36928:36929.

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

Fuzzer: decoder_langfuzz
Job Type: linux_asan_d8_v8_arm_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  reported_errors_end_ <= i in expression-classifier.h
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_d8_v8_arm_dbg&range=36896:36897
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_d8_v8_arm_dbg&range=36928:36929

Minimized Testcase (7.64 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96FdEJMdpxY12WWusqtibtmzAR34HkX039HePrYxSY0yMm7SJq2qa0vrzzeJyheomx2dw6BGzsPT1oo4DZ3kwIxyn8CxmB_szGxiTQcMmrg5nb99VUblFIa3BjJwFiX9oK9YoyXN29ndLERjJy6oa1wMXxXCg

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.
Cc: nikolaos@chromium.org
 Issue 619845  has been merged into this issue.
Project Member

Comment 7 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

Sign in to add a comment