CHECK failure: !isolate->has_scheduled_exception() in builtins-console.cc |
||||||||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=4531783459405824 Fuzzer: mbarbella_js_mutation Job Type: windows_asan_d8_dbg Platform Id: windows Crash Type: CHECK failure Crash Address: Crash State: !isolate->has_scheduled_exception() in builtins-console.cc v8::platform::PrintStackTrace v8::internal::ConsoleCall Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=54217:54218 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4531783459405824 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Jul 20
,
Jul 20
Setting impact and OS labels. A quick look at the stacktrace and the code that triggered the CHECK makes it look like this affects all V8 platforms, so setting accordingly but please update if that is incorrect.
,
Jul 20
,
Jul 21
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 23
,
Jul 23
Removing the ReleaseBlock-Stable label, as this crash only happens in d8 and it's not related to security, and it's behind a flag.
,
Jul 23
,
Jul 23
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/4a28271feeff2dc95ac08c01dc0879facefd58f9 commit 4a28271feeff2dc95ac08c01dc0879facefd58f9 Author: Maya Lekova <mslekova@chromium.org> Date: Mon Jul 23 13:34:50 2018 [async] Improve error handling when running async hooks If an exception is thrown in instrumented async code, for instance await import('non-existing-module') it should be correctly reported by the hooks that run around this code. Also calling ToLocalChecked() on the hook result is wrong if the hook has thrown an exception. Bug: chromium:865892 Change-Id: I5712376fe4426a3e49223d821e4647150887a258 Reviewed-on: https://chromium-review.googlesource.com/1146561 Commit-Queue: Maya Lekova <mslekova@chromium.org> Reviewed-by: Yang Guo <yangguo@chromium.org> Reviewed-by: Benedikt Meurer <bmeurer@chromium.org> Reviewed-by: Dan Elphick <delphick@chromium.org> Cr-Commit-Position: refs/heads/master@{#54610} [modify] https://crrev.com/4a28271feeff2dc95ac08c01dc0879facefd58f9/src/async-hooks-wrapper.cc [modify] https://crrev.com/4a28271feeff2dc95ac08c01dc0879facefd58f9/src/isolate.h [add] https://crrev.com/4a28271feeff2dc95ac08c01dc0879facefd58f9/test/mjsunit/regress/regress-crbug-865892.js
,
Jul 23
,
Jul 24
ClusterFuzz has detected this issue as fixed in range 54609:54610. Detailed report: https://clusterfuzz.com/testcase?key=4531783459405824 Fuzzer: mbarbella_js_mutation Job Type: windows_asan_d8_dbg Platform Id: windows Crash Type: CHECK failure Crash Address: Crash State: !isolate->has_scheduled_exception() in builtins-console.cc v8::platform::PrintStackTrace v8::internal::ConsoleCall Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=54217:54218 Fixed: https://clusterfuzz.com/revisions?job=windows_asan_d8_dbg&range=54609:54610 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4531783459405824 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 24
ClusterFuzz testcase 4531783459405824 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 |
||||||||||
Comment 1 by ClusterFuzz
, Jul 20Owner: mslekova@chromium.org
Status: Assigned (was: Untriaged)