From 03bd05f0e844bf4f952f0ce3cc03887d1c1180d8 Mon Sep 17 00:00:00 2001 From: Vitaly Buka Date: Wed, 18 Aug 2021 19:02:02 -0700 Subject: [PATCH] [sanitizer] Use TMPDIR in Android test TMPDIR was added long time ago, so no need to use EXTERNAL_STORAGE. --- compiler-rt/lib/sanitizer_common/tests/sanitizer_libc_test.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/compiler-rt/lib/sanitizer_common/tests/sanitizer_libc_test.cpp b/compiler-rt/lib/sanitizer_common/tests/sanitizer_libc_test.cpp index 863a433..0e269d6 100644 --- a/compiler-rt/lib/sanitizer_common/tests/sanitizer_libc_test.cpp +++ b/compiler-rt/lib/sanitizer_common/tests/sanitizer_libc_test.cpp @@ -77,11 +77,7 @@ static void temp_file_name(char *buf, size_t bufsize, const char *prefix) { #else const char *tmpdir = "/tmp"; #if SANITIZER_ANDROID - // I don't know a way to query temp directory location on Android without - // going through Java interfaces. The code below is not ideal, but should - // work. May require "adb root", but it is needed for almost any use of ASan - // on Android already. - tmpdir = GetEnv("EXTERNAL_STORAGE"); + tmpdir = GetEnv("TMPDIR"); #endif internal_snprintf(buf, bufsize, "%s/%sXXXXXX", tmpdir, prefix); ASSERT_TRUE(mkstemp(buf)); -- 2.7.4