DCHECK failure in external_backing_store_bytes_[type] >= amount in spaces.cc |
|||||||||||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=4595570392694784 Fuzzer: ochang_js_fuzzer Job Type: linux_d8_dbg Platform Id: linux Crash Type: DCHECK failure Crash Address: Crash State: external_backing_store_bytes_[type] >= amount in spaces.cc Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=54711:54712 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4595570392694784 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Jul 26
,
Jul 26
,
Jul 27
,
Jul 27
Tentatively assigning to mlippautz since the CL author isn't a project member.
,
Jul 28
,
Jul 28
This is a serious security regression. If you are not able to fix this quickly, please revert the change that introduced it. If this doesn't affect a release branch, or has not been properly classified for severity, please update the Security_Impact or Security_Severity labels, and remove the ReleaseBlock label. To disable this altogether, apply ReleaseBlock-NA. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 29
,
Jul 30
This is a regression in a verifier that checks a path that is not yet fully enabled on ToT. The check just makes sure that counters are in a consistent state. The object model is not affected by this.
,
Jul 30
Sounds good, removing the rest of the security labels in that case.
,
Jul 30
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/0640fd58dd848471264960f2e952f5c82e0e9202 commit 0640fd58dd848471264960f2e952f5c82e0e9202 Author: Rodrigo Bruno <rfbpb@google.com> Date: Mon Jul 30 19:08:57 2018 [heap] Fix ThinString promotion counters. From the promotion code point of view, a ThinString being forwarded to an ExternalString is exactly the same as an actual ExternalString being forwarded to its promoted copy. These changes provide a way to disambiguate both scenarios since they are different for external memory accounting. Bug: chromium:867902 Change-Id: I6fd56ee5e0f8900318466108273ab26e936eb439 Reviewed-on: https://chromium-review.googlesource.com/1152975 Commit-Queue: Rodrigo Bruno <rfbpb@google.com> Reviewed-by: Michael Lippautz <mlippautz@chromium.org> Cr-Commit-Position: refs/heads/master@{#54790} [modify] https://crrev.com/0640fd58dd848471264960f2e952f5c82e0e9202/src/heap/heap.cc [modify] https://crrev.com/0640fd58dd848471264960f2e952f5c82e0e9202/src/heap/scavenger-inl.h [modify] https://crrev.com/0640fd58dd848471264960f2e952f5c82e0e9202/test/cctest/heap/test-external-string-tracker.cc
,
Jul 30
,
Jul 31
ClusterFuzz has detected this issue as fixed in range 54789:54790. Detailed report: https://clusterfuzz.com/testcase?key=4595570392694784 Fuzzer: ochang_js_fuzzer Job Type: linux_d8_dbg Platform Id: linux Crash Type: DCHECK failure Crash Address: Crash State: external_backing_store_bytes_[type] >= amount in spaces.cc Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=54711:54712 Fixed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=54789:54790 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4595570392694784 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.
,
Jul 31
ClusterFuzz testcase 4595570392694784 is verified as fixed, so closing issue as verified. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Jul 31
Issue 869312 has been merged into this issue.
,
Jul 31
Automatically applying components based on crash stacktrace and information from OWNERS files. If this is incorrect, please apply the Test-Predator-Wrong-Components label. |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by ClusterFuzz
, Jul 26Labels: Test-Predator-Auto-CC