[compiler-rt] ad06e95 - Reapply "[NFC][sanitizer] Switch to `gnu_get_libc_version` (#108724)" (#108885)
via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 16 14:54:05 PDT 2024
Author: Vitaly Buka
Date: 2024-09-16T14:54:01-07:00
New Revision: ad06e9562a3d2b532240390275c2b35f58004c04
URL: https://github.com/llvm/llvm-project/commit/ad06e9562a3d2b532240390275c2b35f58004c04
DIFF: https://github.com/llvm/llvm-project/commit/ad06e9562a3d2b532240390275c2b35f58004c04.diff
LOG: Reapply "[NFC][sanitizer] Switch to `gnu_get_libc_version` (#108724)" (#108885)
In #108724 `#ifdef` was used instead of `#if`.
This reverts commit 68e4518598d63efa02230f400e50263baccbb8e4.
Added:
Modified:
compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp
Removed:
################################################################################
diff --git a/compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp b/compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp
index 51c1f9ff111de9..21bd2a23f5763c 100644
--- a/compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp
+++ b/compiler-rt/lib/sanitizer_common/sanitizer_linux_libcdep.cpp
@@ -40,6 +40,10 @@
# include <sys/resource.h>
# include <syslog.h>
+# if SANITIZER_GLIBC
+# include <gnu/libc-version.h>
+# endif
+
# if !defined(ElfW)
# define ElfW(type) Elf_##type
# endif
@@ -198,17 +202,11 @@ bool SetEnv(const char *name, const char *value) {
__attribute__((unused)) static bool GetLibcVersion(int *major, int *minor,
int *patch) {
-# ifdef _CS_GNU_LIBC_VERSION
- char buf[64];
- uptr len = confstr(_CS_GNU_LIBC_VERSION, buf, sizeof(buf));
- if (len >= sizeof(buf))
- return false;
- buf[len] = 0;
- static const char kGLibC[] = "glibc ";
- if (internal_strncmp(buf, kGLibC, sizeof(kGLibC) - 1) != 0)
- return false;
- const char *p = buf + sizeof(kGLibC) - 1;
+# if SANITIZER_GLIBC
+ const char *p = gnu_get_libc_version();
*major = internal_simple_strtoll(p, &p, 10);
+ // Caller does not expect anything else.
+ CHECK_EQ(*major, 2);
*minor = (*p == '.') ? internal_simple_strtoll(p + 1, &p, 10) : 0;
*patch = (*p == '.') ? internal_simple_strtoll(p + 1, &p, 10) : 0;
return true;
More information about the llvm-commits
mailing list