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

Issue 777182 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit 15 days ago
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

CHECK failure: !v8::internal::FLAG_enable_slow_asserts || (object->IsFixedDoubleArray()) in obj

Project Member Reported by ClusterFuzz, Oct 22 2017

Issue description

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !v8::internal::FLAG_enable_slow_asserts || (object->IsFixedDoubleArray()) in obj
  cast
  TryCopyElementsHandleFastNumber
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=48722:48723

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

Issue filed automatically.

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

Comment 1 by ClusterFuzz, Oct 22 2017

Labels: Test-Predator-AutoOwner
Owner: cwhan.t...@gmail.com
Status: Assigned (was: Untriaged)
Automatically assigning owner based on suspected regression changelist https://chromium.googlesource.com/v8/v8/+/501127995efd9e8ff4be11e2e6927a3e10597fd1 ([typedarrays] Reduce overheads of TA.p.set).

If this is incorrect, please remove the owner and apply the Test-Predator-Wrong-CLs label.
Cc: verwa...@chromium.org petermarshall@chromium.org
Labels: -Type-Bug-Security -Security_Severity-High OS-Android OS-Chrome OS-Fuchsia OS-iOS OS-Mac OS-Windows Pri-1 Type-Bug-Regression
This crash is triggered only when the given array's length is zero, so I think this is not a security issue. Even though it triggers a wrong type casting, it operates nothing. it is fixed by returning function when the given array length is zero.

WIP CL (https://chromium-review.googlesource.com/c/v8/v8/+/727448) will fix this issue.
Labels: -OS-Android -OS-Windows -OS-iOS -OS-Chrome -OS-Mac -OS-Fuchsia
Agreed this is not a security issue
Project Member

Comment 5 by bugdroid1@chromium.org, Oct 23 2017

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

commit 6241e81c35125aa3e95a0fc83cca6c0131e7695e
Author: Choongwoo Han <cwhan.tunz@gmail.com>
Date: Mon Oct 23 10:34:11 2017

[typedarrays] Fix a wrong type casting in TA.p.set

- Fix a wrong type casting triggered when a given array's length is zero
- Add a regression test case

Bug:  chromium:777182 ,  chromium:768775 
Change-Id: I615b73e9d7bad657c872c96c7a204efe355d8289
Reviewed-on: https://chromium-review.googlesource.com/732865
Reviewed-by: Peter Marshall <petermarshall@chromium.org>
Commit-Queue: Peter Marshall <petermarshall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#48821}
[modify] https://crrev.com/6241e81c35125aa3e95a0fc83cca6c0131e7695e/src/elements.cc
[add] https://crrev.com/6241e81c35125aa3e95a0fc83cca6c0131e7695e/test/mjsunit/es6/regress/regress-777182.js

Project Member

Comment 6 by ClusterFuzz, Oct 24 2017

ClusterFuzz has detected this issue as fixed in range 48820:48821.

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_asan_d8_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !v8::internal::FLAG_enable_slow_asserts || (object->IsFixedDoubleArray()) in obj
  cast
  TryCopyElementsHandleFastNumber
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=48722:48723
Fixed: https://clusterfuzz.com/revisions?job=linux_asan_d8_dbg&range=48820:48821

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

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 7 by ClusterFuzz, Oct 24 2017

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase 5021611463016448 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 8 by sheriffbot@chromium.org, Oct 24 2017

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Labels: -Test-Predator-AutoOwner Test-Predator-Auto-Owner
Project Member

Comment 10 by sheriffbot@chromium.org, Jan 30 2018

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