Issue metadata
Sign in to add a comment
|
DCHECK failure in IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_) in assembler-mips-inl.h |
||||||||||||||||||||||
Issue descriptionDetailed report: https://clusterfuzz.com/testcase?key=5358879130255360 Fuzzer: ochang_js_fuzzer Job Type: linux_asan_d8_v8_mipsel_dbg Platform Id: linux Crash Type: DCHECK failure Crash Address: Crash State: IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_) in assembler-mips-inl.h V8_Dcheck v8::internal::WasmDebugInfo::RedirectToInterpreter Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_v8_mipsel_dbg&range=49947:49948 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5358879130255360 Issue filed automatically. See https://github.com/google/clusterfuzz-tools for more information.
,
Dec 8 2017
,
Dec 8 2017
BTW, I'm not convinced this is a security issue, since the --future flag is required. Fix underway.
,
Dec 9 2017
,
Dec 9 2017
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
,
Dec 9 2017
,
Dec 10 2017
,
Dec 11 2017
Lowering priority - this happens under the --future flag only. Also removing release block flag, same reason
,
Dec 11 2017
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
,
Dec 11 2017
,
Dec 12 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/a9d9001810aef2b3ad9487b4e9cb5ae766381abc commit a9d9001810aef2b3ad9487b4e9cb5ae766381abc Author: Mircea Trofin <mtrofin@chromium.org> Date: Tue Dec 12 15:02:23 2017 [wasm] Fix up mips & WasmCodeManager Corrected mips code generation for WASM_CALL and JS_TO_WASM_CALL. The logic for fetching raw call sites needed changing, too, in light of mips' 2-instruction calls, where using target_address_address is incorrect. The CL adds platform-specific accessors. Bug: chromium:793292 chromium:793282 Change-Id: I879ea6bffdad60791d88a6f5ea15087cdcd3f2e9 Reviewed-on: https://chromium-review.googlesource.com/818460 Reviewed-by: Ben Titzer <titzer@chromium.org> Reviewed-by: Ivica Bogosavljevic <ivica.bogosavljevic@mips.com> Commit-Queue: Mircea Trofin <mtrofin@chromium.org> Cr-Commit-Position: refs/heads/master@{#50037} [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/compiler/mips/code-generator-mips.cc [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/mips/assembler-mips-inl.h [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/mips64/assembler-mips64-inl.h [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/wasm/wasm-code-manager.cc [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/wasm/wasm-code-manager.h [modify] https://crrev.com/a9d9001810aef2b3ad9487b4e9cb5ae766381abc/src/wasm/wasm-serialization.cc
,
Dec 12 2017
,
Dec 13 2017
ClusterFuzz has detected this issue as fixed in range 50036:50037. Detailed report: https://clusterfuzz.com/testcase?key=5358879130255360 Fuzzer: ochang_js_fuzzer Job Type: linux_asan_d8_v8_mipsel_dbg Platform Id: linux Crash Type: DCHECK failure Crash Address: Crash State: IsCodeTarget(rmode_) || IsRuntimeEntry(rmode_) in assembler-mips-inl.h V8_Dcheck v8::internal::WasmDebugInfo::RedirectToInterpreter Sanitizer: address (ASAN) Regressed: https://clusterfuzz.com/revisions?job=linux_asan_d8_v8_mipsel_dbg&range=49947:49948 Fixed: https://clusterfuzz.com/revisions?job=linux_asan_d8_v8_mipsel_dbg&range=50036:50037 Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=5358879130255360 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.
,
Dec 13 2017
,
Dec 13 2017
ClusterFuzz testcase 5358879130255360 is verified as fixed, so closing issue as verified. If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
,
Dec 15 2017
,
Dec 15 2017
This bug requires manual review: M64 has already been promoted to the beta branch, so this requires manual review Please contact the milestone owner if you have questions. Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), abdulsyed@(Desktop) For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Dec 18 2017
Regression and fix range are both in 65.
,
Mar 21 2018
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
,
Mar 27 2018
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by elawrence@chromium.org
, Dec 8 2017Owner: mtrofin@chromium.org