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

Issue 679202 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 1
Type: Bug



Sign in to add a comment

!v8::internal::FLAG_enable_slow_asserts || (object->IsJSObject()) in objects-inl

Project Member Reported by ClusterFuzz, Jan 8 2017

Issue description

Cc: msrchandra@chromium.org
Labels: Test-Predator-Wrong
Owner: bmeu...@chromium.org
Status: Assigned (was: Untriaged)
Find it did not provide any possible suspects.
From CL assigning to the concern owner --
https://chromium.googlesource.com/v8/v8/+log/c7a6790050023ff22515233f820fe5c8dc104a11..e56f265f6d41fadbcea2be65d9b573bad8b4709d?pretty=fuller

@bmeurer -- Could you please look into the isue, kindly re-assign if this is not related to your changes.
Thank You.
Cc: ishell@chromium.org jkummerow@chromium.org
Status: Started (was: Assigned)
Project Member

Comment 4 by bugdroid1@chromium.org, Jan 9 2017

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

commit 5f418c8a2d95620c8d328e1357e9373c74e685e6
Author: bmeurer <bmeurer@chromium.org>
Date: Mon Jan 09 08:47:43 2017

[crankshaft] Properly deal with null prototype.

Don't assume that the prototype of an object is always a JSObject when
inlining the known receiver map case for abstract relational comparison.

BUG= chromium:679202 
R=ishell@chromium.org

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

[modify] https://crrev.com/5f418c8a2d95620c8d328e1357e9373c74e685e6/src/crankshaft/hydrogen.cc
[add] https://crrev.com/5f418c8a2d95620c8d328e1357e9373c74e685e6/test/mjsunit/regress/regress-crbug-679202.js

Project Member

Comment 5 by bugdroid1@chromium.org, Jan 9 2017

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

commit 5f418c8a2d95620c8d328e1357e9373c74e685e6
Author: bmeurer <bmeurer@chromium.org>
Date: Mon Jan 09 08:47:43 2017

[crankshaft] Properly deal with null prototype.

Don't assume that the prototype of an object is always a JSObject when
inlining the known receiver map case for abstract relational comparison.

BUG= chromium:679202 
R=ishell@chromium.org

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

[modify] https://crrev.com/5f418c8a2d95620c8d328e1357e9373c74e685e6/src/crankshaft/hydrogen.cc
[add] https://crrev.com/5f418c8a2d95620c8d328e1357e9373c74e685e6/test/mjsunit/regress/regress-crbug-679202.js

Cc: habl...@google.com
Labels: -OS-Linux Merge-Request-56 Merge-Request-55 OS-All
Status: Fixed (was: Started)
Cc: -habl...@google.com hablich@chromium.org
Labels: -Merge-Request-56 Merge-Approved-56
Let's wait for some Canary coverage and than merge it to beta. I don't think there will be a another 55 push. Let's hold off merging it for now.
Project Member

Comment 8 by ClusterFuzz, Jan 10 2017

ClusterFuzz has detected this issue as fixed in range 42122:42123.

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

Fuzzer: decoder_langfuzz
Job Type: linux_asan_d8_v8_arm64_dbg
Platform Id: linux

Crash Type: CHECK failure
Crash Address: 
Crash State:
  !v8::internal::FLAG_enable_slow_asserts || (object->IsJSObject()) in objects-inl
  
Sanitizer: address (ASAN)

Regressed: V8: r30851:30852
Fixed: V8: r42122:42123

Minimized Testcase (6.92 Kb): https://cluster-fuzz.appspot.com/download/AMIfv97ORYYCyB8vKQxM7ympIUrG2tgnBkxGyOefnHcHHvSI8ZzFODhSjJp1fSgvoocSO32HHonZdDT__-r8oLRMBI7LaJRDi0ZcRLiwc-axbK4T8XWuW-_RJnF_lN9JgEql_CJmswTw-lzZeS7ZAu1aXOfVvjC-eLBs41mA_mR1-e02IKpB6IK2AsakdjowuO6sdf0tHAaM_JRqgRSDVNWFYqQHk9h-o1f1CtCDIo6rRtdtXUDokb_L3hSX3zdrkd7PEouRxZZ3YWM2r7IIUs4adkNhYRyUBQMu2tdDoJ5mHHfIXIbsKdAImpqxrLDNqx79pCGoSsisuFT4T06RYuMxyWX_K0iC8Tesejyd9c9MDZgoXNDGxQ_G2ch4jvIud5LF256AWfDripRKGPSySw3_VoyBMBDfeg?testcase_id=5492266878894080

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.
Your change is approved for M56,  if all looks good please merge the CL ASAP.
We are planning to cut Beta RC today 01/10 at 4.00 PM PST.
Labels: -Merge-Approved-56 merge-merged-56
Labels: -Merge-Request-55 Merge-Rejected-55

Sign in to add a comment