<div dir="ltr">makes sense, thanks!  </div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 11, 2016 at 10:13 AM, Alexey Samsonov <span dir="ltr"><<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I suspect that's because of different implementations of printf, strcasestr, strstr in libc. These tests failed only on Android and Apple platforms, where (apparently) these functions call strlen(), thus triggering ASan interceptor.</div><div class="gmail_extra"><div><div class="h5"><br><div class="gmail_quote">On Fri, Mar 11, 2016 at 9:08 AM, Kostya Serebryany <span dir="ltr"><<a href="mailto:kcc@google.com" target="_blank">kcc@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">do you know why this was not caught during manual testing before commit? </div><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 10, 2016 at 9:04 PM, Alexey Samsonov via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: samsonov<br>
Date: Thu Mar 10 23:04:49 2016<br>
New Revision: 263195<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=263195&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=263195&view=rev</a><br>
Log:<br>
Fix ASan test cases after r263177<br>
<br>
Modified:<br>
    compiler-rt/trunk/test/asan/TestCases/printf-2.c<br>
    compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c<br>
    compiler-rt/trunk/test/asan/TestCases/strstr-2.c<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/printf-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/printf-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/printf-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/printf-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/printf-2.c Thu Mar 10 23:04:49 2016<br>
@@ -1,9 +1,9 @@<br>
 // RUN: %clang_asan -O2 %s -o %t<br>
-// We need replace_str=0 and replace_intrin=0 to avoid reporting errors in<br>
-// strlen() and memcpy() called by printf().<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0:check_printf=1 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0:check_printf=0 %run %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s<br>
-// RUN: %env_asan_opts=replace_str=0:replace_intrin=0 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
+// We need replace_str=0, intercept_strlen=0 and replace_intrin=0 to avoid<br>
+// reporting errors in strlen() and memcpy() called by printf().<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0:check_printf=1 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0:check_printf=0 %run %t 2>&1 | FileCheck --check-prefix=CHECK-OFF %s<br>
+// RUN: %env_asan_opts=replace_str=0:intercept_strlen=0:replace_intrin=0 not %run %t 2>&1 | FileCheck --check-prefix=CHECK-ON %s<br>
<br>
 // FIXME: printf is not intercepted on Windows yet.<br>
 // XFAIL: win32<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/strcasestr-2.c Thu Mar 10 23:04:49 2016<br>
@@ -3,7 +3,7 @@<br>
<br>
 // Test intercept_strstr asan option<br>
 // Disable other interceptors because strlen may be called inside strcasestr<br>
-// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false %run %t 2>&1<br>
+// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false:intercept_strlen=false %run %t 2>&1<br>
<br>
 // There's no interceptor for strcasestr on Windows<br>
 // XFAIL: win32<br>
<br>
Modified: compiler-rt/trunk/test/asan/TestCases/strstr-2.c<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strstr-2.c?rev=263195&r1=263194&r2=263195&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/TestCases/strstr-2.c?rev=263195&r1=263194&r2=263195&view=diff</a><br>
==============================================================================<br>
--- compiler-rt/trunk/test/asan/TestCases/strstr-2.c (original)<br>
+++ compiler-rt/trunk/test/asan/TestCases/strstr-2.c Thu Mar 10 23:04:49 2016<br>
@@ -3,7 +3,7 @@<br>
<br>
 // Test intercept_strstr asan option<br>
 // Disable other interceptors because strlen may be called inside strstr<br>
-// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false %run %t 2>&1<br>
+// RUN: %env_asan_opts=intercept_strstr=false:replace_str=false:intercept_strlen=false %run %t 2>&1<br>
<br>
 #include <assert.h><br>
 #include <string.h><br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</font></span></div>
</blockquote></div><br></div>