From 5cb378fab3bedbaeef708ae725bdccb813d48519 Mon Sep 17 00:00:00 2001 From: Vy Nguyen Date: Thu, 5 Nov 2020 00:27:38 -0500 Subject: [PATCH] [sanitizers] Remove the test case involving `new int[0]` Bionic doesn't acutally allocate any memory in this case, so there won't be a leak on Android. Differential Revision: https://reviews.llvm.org/D90821 --- compiler-rt/test/asan/TestCases/leaks.cpp | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/compiler-rt/test/asan/TestCases/leaks.cpp b/compiler-rt/test/asan/TestCases/leaks.cpp index 9c076dd..1207818 100644 --- a/compiler-rt/test/asan/TestCases/leaks.cpp +++ b/compiler-rt/test/asan/TestCases/leaks.cpp @@ -14,6 +14,14 @@ int *t; __attribute__((noopt)) void leak(int n) { +#if defined(__ANDROID__) || defined(__BIONIC__) + // Bionic does not acutally allocate when n==0, hence + // there would not be a leak. + // Re-adjust n so the test can pass. + if (n == 0) + n = 1; +#endif + // Repeat few times to make sure that at least one pointer is // not somewhere on the stack. for (int i = 0; i < 10; ++i) { -- 2.7.4