[1.6>master] [MERGE #3129 @Penguinwizzard] Fix a few issues blocking …

…ASAN runs Merge pull request #3129 from Penguinwizzard:fix_asan_ci Disable one test in ASAN due to a known issue with modules in ch (bug #2735), disable one test on xplat due to wasm calling convention issues (bug #2378), keep a loop from accessing data beyond an array bound in the pal code, and fix the casing on one test file.
chakra-core · Jun 9, 2017 · a73911f · a73911f
2 parents d71f43e + 07bb54c
commit a73911f
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 2 deletions.
diff --git a/pal/src/cruntime/wchar.cpp b/pal/src/cruntime/wchar.cpp
@@ -1245,7 +1245,7 @@ PAL_wmemcmp(
     const     size_t *num1     = (const size_t*)(string1);
     const     size_t *num2     = (const size_t*)(string2);
 
-    while( (count > blockSize * wi) && num1[wi] == num2[wi] ) ++wi;
+    while( (count > blockSize * (wi + 1)) && num1[wi] == num2[wi] ) ++wi;
 
     for (i = blockSize * wi; i < count; ++i)
     {

diff --git a/test/es6/rlexe.xml b/test/es6/rlexe.xml
@@ -1415,7 +1415,7 @@
   <default>
     <files>bug_issue_3076.js</files>
     <compile-flags>-force:deferparse</compile-flags>
-    <tags>BugFix</tags>
+    <tags>BugFix,exclude_sanitize_address</tags>
   </default>
 </test>
 <test>

diff --git a/test/typedarray/floathelperaccess.js → test/typedarray/FloatHelperAccess.js b/test/typedarray/floathelperaccess.js → test/typedarray/FloatHelperAccess.js
diff --git a/test/wasm/rlexe.xml b/test/wasm/rlexe.xml
@@ -89,6 +89,7 @@
     <files>basic.js</files>
     <baseline>basic.baseline</baseline>
     <compile-flags>-wasm</compile-flags>
+    <tags>exclude_xplat</tags>
   </default>
 </test>
 <test>