chakra-core / chakra-core/ChakraCore

Assertion Failure: (added || failedToUpdateCompatibleUpperBoundCheck) in ChakraCore/lib/Backend/GlobOptArrays.cpp, line 1267

Open
#6,917 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
9.3k
Forks
1.2k
PR merge metrics
No merged PRs in 30d

Description

## Version
commit id: c3ead3f8a6e0bb8e32e043adc091c68cba5935e9

## Platform
Ubuntu 20.04.5 LTS (Linux 5.4.0-144-generic x86_64)

## Build
- Debug Mode

```
./build.sh --debug --static
```

## PoC
```
for (let i = 0; i < 60; i++) {
const arr = [];
for (let k = 2; k > 2; k = k ^ -2) {
for (let m = -2; m > -2; m = m ^ 2) {
let q = 2;
for (let n = m; n < q; n = n + 128) {
q--;
arr[q] = n;
}
}
arr[128] = 2;
}
}
```

## Execution steps & Output
```
./ch --maxinterpretcount:10 --maxsimplejitruncount:100 -bgjit- poc.js
ASSERTION 432771: (/home/wjm/ChakraCore/lib/Backend/GlobOptArrays.cpp, line 1267) added || failedToUpdateCompatibleUpperBoundCheck
Failure: (added || failedToUpdateCompatibleUpperBoundCheck)
Signal: SIGILL (Illegal instruction)
```
## Backtrace
```
(lldb) bt
* thread #1, name = 'ch', stop reason = signal SIGILL: illegal instruction operand
* frame #0: 0x0000555556bbd9cb ch`GlobOpt::ArraySrcOpt::DoUpperBoundCheck(this=0x00007fffffff80b8) at GlobOptArrays.cpp:1267:17
frame #1: 0x0000555556bba46b ch`GlobOpt::ArraySrcOpt::DoExtractBoundChecks(this=0x00007fffffff80b8) at GlobOptArrays.cpp:843:9
frame #2: 0x0000555556bc15a9 ch`GlobOpt::ArraySrcOpt::Optimize(this=0x00007fffffff80b8) at GlobOptArrays.cpp:1896:13
frame #3: 0x0000555556b5fae3 ch`GlobOpt::OptArraySrc(this=0x00007fffffff9820, instrRef=0x00007fffffff8698, src1Val=0x00007fffffff8530, src2Val=0x00007fffffff8528) at GlobOpt.cpp:13452:17
frame #4: 0x0000555556b49f2e ch`GlobOpt::OptInstr(this=0x00007fffffff9820, instr=0x00007fffffff8698, isInstrRemoved=0x00007fffffff868e) at GlobOpt.cpp:2563:5
frame #5: 0x0000555556b46dce ch`GlobOpt::OptBlock(this=0x00007fffffff9820, block=0x00007ff7e7c07508) at GlobOpt.cpp:521:27
frame #6: 0x0000555556b458a7 ch`GlobOpt::ForwardPass(this=0x00007fffffff9820) at GlobOpt.cpp:402:15
frame #7: 0x0000555556b44ab3 ch`GlobOpt::Optimize(this=0x00007fffffff9820) at GlobOpt.cpp:212:15
frame #8: 0x0000555556b262ab ch`Func::TryCodegen(this=0x00007fffffff9ed0) at Func.cpp:457:21
frame #9: 0x0000555556b25ae0 ch`Func::Codegen(alloc=0x00007fffffffa490, workItem=0x00007ff7e7bf5030, threadContextInfo=0x0000555557eb70d8, scriptContextInfo=0x0000555557ee9078, outputData=0x00007fffffffa8d0, epInfo=0x00007ff7e7c64200, runtimeInfo=0x0000000000000000, polymorphicInlineCacheInfo=0x00007ff7e7c3f300, codeGenAllocators=0x0000555557ef61a8, codeGenProfiler=0x0000000000000000, isBackgroundJIT=false) at Func.cpp:325:18
frame #10: 0x000055555690ef90 ch`NativeCodeGenerator::CodeGen(this=0x0000555557eea848, pageAllocator=0x0000555557eb7460, workItemData=0x0000555557ef6020, jitWriteData=0x00007fffffffa8d0, foreground=true, epInfo=0x00007ff7e7c64200) at NativeCodeGenerator.cpp:890:9
frame #11: 0x0000555556910c28 ch`NativeCodeGenerator::CodeGen(this=0x0000555557eea848, pageAllocator=0x0000555557eb7460, workItem=0x0000555557ef5ff8, foreground=true) at NativeCodeGenerator.cpp:1007:5
frame #12: 0x00005555569139a7 ch`NativeCodeGenerator::Process(this=0x0000555557eea848, job=0x0000555557ef6000, threadData=0x0000000000000000) at NativeCodeGenerator.cpp:1895:13
frame #13: 0x00005555569b83d4 ch`JsUtil::ForegroundJobProcessor::Process(job=0x0000555557ef6000) at Jobs.cpp:426:36
frame #14: 0x0000555556922bc3 ch`void JsUtil::ForegroundJobProcessor::PrioritizeJobAndWait(this=0x0000555557ea2ce8, manager=0x0000555557eea848, holder=0x00007ff7e7c64200, function=0x0000000000000000) at Jobs.inl:207:36
frame #15: 0x0000555556918804 ch`void JsUtil::JobProcessor::PrioritizeJobAndWait(this=0x0000555557ea2ce8, manager=0x0000555557eea848, holder=0x00007ff7e7c64200, function=0x0000000000000000) at Jobs.inl:81:58
frame #16: 0x000055555690d4fc ch`NativeCodeGenerator::GenerateLoopBody(this=0x0000555557eea848, fn=0x00007ff7e7c47000, loopHeader=0x00007ff7e7c430f0, entryPoint=0x00007ff7e7c64200, localCount=21, localSlots=0x00007fffffffc0f0) at NativeCodeGenerator.cpp:711:22
frame #17: 0x00005555568157dd ch`GenerateLoopBody(nativeCodeGen=0x0000555557eea848, fn=0x00007ff7e7c47000, loopHeader=0x00007ff7e7c430f0, info=0x00007ff7e7c64200, localCount=21, localSlots=0x00007fffffffc0f0) at BackendApi.cpp:88:20
frame #18: 0x0000555555fb220b ch`Js::InterpreterStackFrame::DoLoopBodyStart(this=0x00007fffffffbf90, loopNumber=1, layoutSize=SmallLayout, doProfileLoopCheck=true, isFirstIteration=false) at InterpreterStackFrame.cpp:6277:17
frame #19: 0x0000555555fb5997 ch`void Js::InterpreterStackFrame::LoopBodyStart(this=0x00007fffffffbf90, loopNumber=1, layoutSize=SmallLayout, isFirstIteration=false) at InterpreterStackFrame.cpp:5994:9
frame #20: 0x0000555556002d38 ch`unsigned char const* Js::InterpreterStackFrame::OP_ProfiledLoopBodyStart<(Js::LayoutSize)0, false>(this=0x00007fffffffbf90, loopId=1) at InterpreterStackFrame.cpp:5902:9
frame #21: 0x0000555555fcd716 ch`unsigned char const* Js::InterpreterStackFrame::OP_ProfiledLoopBodyStart<(Js::LayoutSize)0, false>(this=0x00007fffffffbf90, ip="\U00000011\U00000003") at InterpreterStackFrame.cpp:5774:16
frame #22: 0x0000555555e704b1 ch`Js::InterpreterStackFrame::ProcessUnprofiled(this=0x00007fffffffbf90) at InterpreterHandler.inl:52:3
frame #23: 0x0000555555e522be ch`Js::InterpreterStackFrame::Process(this=0x00007fffffffbf90) at InterpreterStackFrame.cpp:3495:22
frame #24: 0x0000555555e50dd3 ch`Js::InterpreterStackFrame::InterpreterHelper(function=0x00007ff7e7c76730, args=ArgumentReader @ 0x00007fffffffc520, returnAddress=0x00007ff7e7bd0fa2, addressOfReturnAddress=0x00007fffffffc568, asmJsReturn=0x0000000000000000) at InterpreterStackFrame.cpp:2153:40
frame #25: 0x0000555555e4feb0 ch`Js::InterpreterStackFrame::InterpreterThunk(layout=0x00007fffffffc580) at InterpreterStackFrame.cpp:1833:16
frame #26: 0x00007ff7e7bd0fa2
frame #27: 0x00005555564a37de ch`amd64_CallFunction at JavascriptFunctionA.S:100
frame #28: 0x00005555561d7a4b ch`void* Js::JavascriptFunction::CallFunction(function=0x00007ff7e7c76730, entryPoint=(ch`NativeCodeGenerator::CheckCodeGenThunk(Js::RecyclableObject*, Js::CallInfo, ...)), args=Arguments @ 0x00007fffffffc788, useLargeArgCount=false)(Js::RecyclableObject*, Js::CallInfo, ...), Js::Arguments, bool) at JavascriptFunction.cpp:1364:16
frame #29: 0x00005555561cf2f4 ch`Js::JavascriptFunction::CallRootFunctionInternal(obj=0x00007ff7e7c76730, args=Arguments @ 0x00007fffffffc800, scriptContext=0x0000555557ee9078, inScript=true) at JavascriptFunction.cpp:772:24
frame #30: 0x00005555561cf10c ch`Js::JavascriptFunction::CallRootFunction(obj=0x00007ff7e7c76730, args=, scriptContext=0x0000555557ee9078, inScript=true) at JavascriptFunction.cpp:717:15
frame #31: 0x00005555561cf0b1 ch`Js::JavascriptFunction::CallRootFunction(this=0x00007ff7e7c76730, args=, scriptContext=0x0000555557ee9078, inScript=true) at JavascriptFunction.cpp:832:16
frame #32: 0x0000555555894a8e ch`RunScriptCore(this=0x00007fffffffcbd0, scriptContext=0x0000555557ee9078, _actionEntryPopper=0x00007fffffffcbb0)::$_85::operator()(Js::ScriptContext*, TTD::TTDJsRTActionResultAutoRecorder&) const at Jsrt.cpp:3705:49
frame #33: 0x0000555555894624 ch`_JsErrorCode ContextAPIWrapper(this=0x00007fffffffcb68, scriptContext=0x0000555557ee9078)::$_85)::'lambda'(Js::ScriptContext*)::operator()(Js::ScriptContext*) const at JsrtInternal.h:237:16
frame #34: 0x0000555555893fc4 ch`_JsErrorCode ContextAPIWrapper_Core(RunScriptCore(void*, unsigned char const*, unsigned long, LoadScriptFlag, unsigned long, char16_t const*, bool, _JsParseScriptAttributes, bool, void**)::$_85)::'lambda'(Js::ScriptContext*)>(fn=(anonymous class) @ 0x00007fffffffcb68)::$_85) at JsrtInternal.h:192:23
frame #35: 0x00005555558608f6 ch`_JsErrorCode ContextAPIWrapper(fn=(anonymous class) @ 0x00007fffffffcbd0)::$_85) at JsrtInternal.h:235:27
frame #36: 0x00005555558607fb ch`RunScriptCore(scriptSource=0x00007ff7e7c44000, script="for (let i = 0; i < 60; i++) {\n const arr = [];\n for (let k = 2; k > 2; k = k ^ -2) {\n for (let m = -2; m > -2; m = m ^ 2) {\n let q = 2;\n for (let n = m; n < q; n = n + 128) {\n q--;\n arr[q] = n;\n }\n }\n arr[128] = 2;\n }\n}\n// CRASH INFO\n// ==========\n// TERMSIG: 4\n// STDERR:\n// ASSERTION 72586: (/home/wjm/ChakraCore/lib/Backend/GlobOptArrays.cpp, line 1267) added || failedToUpdateCompatibleUpperBoundCheck\n// Failure: (added || failedToUpdateCompatibleUpperBoundCheck)\n// STDOUT:\n// ARGS: /home/wjm/ChakraCore/out/Debug/ch --maxinterpretcount:10 --maxsimplejitruncount:100 -bgjit- -reprl fuzzcode.js\n// EXECUTION TIME: 67 ms\n\n/*\n\nTitle: Assertion Failure: \n\n## Version\ncommit id: c3ead3f8a6e0bb8e32e043adc091c68cba5935e9\n\n## Platform\nUbuntu 20.04.5 LTS (Linux 5.4.0-144-generic x86_64)\n\n## Build\n- Debug Mode\n\n```\n./build.sh --debug --static\n```\n\n## PoC\n```\n\n```\n\n## Execution steps & Output\n```\n./ch --maxinterpretcount:10 --max"..., cb=1093, loadScriptFlag=LoadScriptFlag_Utf8Source | LoadScriptFlag_ExternalArrayBuffer, sourceContext=0, sourceUrl=u"/home/wjm/DiTing-pocs/chakra/bug32_upperBoundCheck.js", parseOnly=false, parseAttributes=JsParseScriptAttributeNone, isSourceModule=false, result=0x0000000000000000) at Jsrt.cpp:3656:12
frame #37: 0x0000555555862f6e ch`::JsRun(JsValueRef, JsSourceContext, JsValueRef, JsParseScriptAttributes, JsValueRef *) [inlined] CompileRun(scriptVal=0x00007ff7e7c44000, sourceContext=0, sourceUrl=0x00007ff7e7c71cf0, parseAttributes=JsParseScriptAttributeNone, result=0x0000000000000000, parseOnly=false) at Jsrt.cpp:5019:12
frame #38: 0x0000555555862db9 ch`::JsRun(scriptVal=0x00007ff7e7c44000, sourceContext=0, sourceUrl=0x00007ff7e7c71cf0, parseAttributes=JsParseScriptAttributeNone, result=0x0000000000000000) at Jsrt.cpp:5041
frame #39: 0x0000555555787293 ch`ChakraRTInterface::JsRun(script=0x00007ff7e7c44000, sourceContext=0, sourceUrl=0x00007ff7e7c71cf0, parseAttributes=JsParseScriptAttributeNone, result=0x0000000000000000) at ChakraRtInterface.h:487:179
frame #40: 0x0000555555784924 ch`RunScript(fileName="bug32_upperBoundCheck.js", fileContents="for (let i = 0; i < 60; i++) {\n const arr = [];\n for (let k = 2; k > 2; k = k ^ -2) {\n for (let m = -2; m > -2; m = m ^ 2) {\n let q = 2;\n for (let n = m; n < q; n = n + 128) {\n q--;\n arr[q] = n;\n }\n }\n arr[128] = 2;\n }\n}\n// CRASH INFO\n// ==========\n// TERMSIG: 4\n// STDERR:\n// ASSERTION 72586: (/home/wjm/ChakraCore/lib/Backend/GlobOptArrays.cpp, line 1267) added || failedToUpdateCompatibleUpperBoundCheck\n// Failure: (added || failedToUpdateCompatibleUpperBoundCheck)\n// STDOUT:\n// ARGS: /home/wjm/ChakraCore/out/Debug/ch --maxinterpretcount:10 --maxsimplejitruncount:100 -bgjit- -reprl fuzzcode.js\n// EXECUTION TIME: 67 ms\n\n/*\n\nTitle: Assertion Failure: \n\n## Version\ncommit id: c3ead3f8a6e0bb8e32e043adc091c68cba5935e9\n\n## Platform\nUbuntu 20.04.5 LTS (Linux 5.4.0-144-generic x86_64)\n\n## Build\n- Debug Mode\n\n```\n./build.sh --debug --static\n```\n\n## PoC\n```\n\n```\n\n## Execution steps & Output\n```\n./ch --maxinterpretcount:10 --max"..., fileLength=1093, fileContentsFinalizeCallback=(ch`WScriptJsrt::FinalizeFree(void*) at WScriptJsrt.cpp:217), bufferValue=0x0000000000000000, fullPath="/home/wjm/DiTing-pocs/chakra/bug32_upperBoundCheck.js", parserStateCache=0x0000000000000000)(void*), void*, char*, void*) at ch.cpp:451:25
frame #41: 0x00005555557863f0 ch`ExecuteTest(fileName="bug32_upperBoundCheck.js") at ch.cpp:917:13
frame #42: 0x00005555557864ac ch`ExecuteTestWithMemoryCheck(fileName="bug32_upperBoundCheck.js") at ch.cpp:967:10
frame #43: 0x0000555555786d7a ch`main(argc=5, c_argv=0x00007fffffffd638) at ch.cpp:1274:20
frame #44: 0x00007ffff778d1e2 libc.so.6`__libc_start_main + 242
frame #45: 0x0000555555783b7e ch`_start + 46

```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.