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

Issue 875847 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Aug 21
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

DCHECK failure in obj->IsExternalString() in heap.cc

Project Member Reported by ClusterFuzz, Aug 20

Issue description

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_d8_dbg
Platform Id: linux

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  obj->IsExternalString() in heap.cc
  
Sanitizer: address (ASAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_d8_dbg&range=54711:54712

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

Issue filed automatically.

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

Comment 1 by sheriffbot@chromium.org, Aug 20

Labels: Pri-1
Project Member

Comment 2 by ClusterFuzz, Aug 20

Cc: rfbpb@google.com
Labels: Test-Predator-Auto-CC
Automatically adding ccs based on suspected regression changelists:

Reland^2 "Added External Strings to external memory accounting". by rfbpb@google.com - https://chromium.googlesource.com/v8/v8/+/82632ed5e47e08036365cb568ed3f9db79a567d1

If this is incorrect, please let us know why and apply the Test-Predator-Wrong-CLs label.
Cc: mlippautz@chromium.org
Cc: -mlippautz@chromium.org
Components: -Blink>JavaScript Blink>JavaScript>GC
Owner: mlippautz@chromium.org
Status: Started (was: Untriaged)
Labels: Security_Impact-Head M-70
Project Member

Comment 6 by bugdroid1@chromium.org, Aug 21

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

commit cd2126feddba2027257fa6474a9982ae5c689c32
Author: Rodrigo Bruno <rfbpb@google.com>
Date: Tue Aug 21 07:09:22 2018

[heap] split ExternalStringTable verify into new space and full (new and old)

Bug:  chromium:875847 
Change-Id: I1412b25c667dc007dd4f605ec9abe20670e75836
Reviewed-on: https://chromium-review.googlesource.com/1181567
Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Commit-Queue: Rodrigo Bruno <rfbpb@google.com>
Cr-Commit-Position: refs/heads/master@{#55243}
[modify] https://crrev.com/cd2126feddba2027257fa6474a9982ae5c689c32/src/heap/heap.cc
[modify] https://crrev.com/cd2126feddba2027257fa6474a9982ae5c689c32/src/heap/heap.h

Status: Fixed (was: Started)
Project Member

Comment 8 by ClusterFuzz, Aug 21

ClusterFuzz has detected this issue as fixed in range 55242:55243.

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

Fuzzer: ochang_js_fuzzer
Job Type: linux_d8_dbg
Platform Id: linux

Crash Type: DCHECK failure
Crash Address: 
Crash State:
  obj->IsExternalString() in heap.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=55242:55243

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

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, Aug 21

Labels: ClusterFuzz-Verified
Status: Verified (was: Fixed)
ClusterFuzz testcase 6341925066768384 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 10 by sheriffbot@chromium.org, Aug 21

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 11 by sheriffbot@chromium.org, Nov 27

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