New issue
Advanced search Search tips

Issue 879560 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Aug 31
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug



Sign in to add a comment

DCHECK failure in (pointer_) != nullptr in utils.h

Project Member Reported by ClusterFuzz, Aug 31

Issue description

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

Fuzzer: mbarbella_js_mutation
Job Type: windows_asan_d8_dbg
Platform Id: windows

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  (pointer_) != nullptr in utils.h
  v8::platform::PrintStackTrace
  v8::internal::compiler::SimplifiedLowering::ToNumericCode
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=55529:55530

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Aug 31

Labels: Test-Predator-Auto-Owner
Owner: bmeu...@chromium.org
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/v8/v8/+/c7b15fb7cdb6f483013832e5834d1290fd015a26 ([turbofan] Lower to JSToNumeric to JSToNumber if possible.).

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.
Project Member

Comment 2 by sheriffbot@chromium.org, Aug 31

Labels: Pri-1
Status: Started (was: Assigned)
Labels: -Restrict-View-SecurityTeam -Security_Severity-High
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 31

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

commit b1bd6beb4e6cb8f1cc8fb8f4fc6e32a9382561ac
Author: Benedikt Meurer <bmeurer@chromium.org>
Date: Fri Aug 31 14:58:25 2018

[turbofan] Fix typo flushed out by recent CL.

Bug:  chromium:879560 
Change-Id: Ia2d2699851358641d50f9997875810f8cb1100ff
Reviewed-on: https://chromium-review.googlesource.com/1199742
Reviewed-by: Maya Lekova <mslekova@chromium.org>
Commit-Queue: Benedikt Meurer <bmeurer@chromium.org>
Cr-Commit-Position: refs/heads/master@{#55553}
[modify] https://crrev.com/b1bd6beb4e6cb8f1cc8fb8f4fc6e32a9382561ac/src/compiler/simplified-lowering.cc
[add] https://crrev.com/b1bd6beb4e6cb8f1cc8fb8f4fc6e32a9382561ac/test/mjsunit/regress/regress-crbug-879560.js

bmeurer - For completeness, what happens on a release build where there is no DCHECK()?  Do we hit some other memory issue later on, or is it just an incorrect DCHECK()?  If its the later, we can move this out of the security bug category.
Cc: neis@chromium.org
Components: -Blink>JavaScript Blink>JavaScript>Compiler
Labels: -Type-Bug-Security OS-Linux Type-Bug
Status: Fixed (was: Started)
It'll just crash due to dereferencing a nullptr.
Project Member

Comment 8 by ClusterFuzz, Sep 1

ClusterFuzz has detected this issue as fixed in range 55552:55553.

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

Fuzzer: mbarbella_js_mutation
Job Type: windows_asan_d8_dbg
Platform Id: windows

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  (pointer_) != nullptr in utils.h
  v8::platform::PrintStackTrace
  v8::internal::compiler::SimplifiedLowering::ToNumericCode
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=55529:55530
Fixed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=55552:55553

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

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 9 by ClusterFuzz, Sep 1

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

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

Sign in to add a comment