sanitizers: Do not include crypt.h if SANITIZER_INTERCEPT_CRYPT_R is undef

[thakis]

Summary

sanitizer_intercept_overriders.h might override SANITIZER_INTERCEPT_CRYPT_R to
be undefined. There's no need to require crypt.h in that case.

(The motivation is that crypt() moved from glibc into its own package at some
point, which makes intercepting it and building with a single sysroot that
supports both pre-bullseye and post-bullseye a bit hairy.)

[thakis]

compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.h

21 ↗

(On Diff #433065)

(not sure if adding this include here is proper?)

[thakis]

compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.h

21 ↗

(On Diff #433065)

Ah hm, looks like it isn't: sanitizer_platform_interceptors.h includes sanitizer_platform_limits_posix.h.

Any suggestions on how prevent the crypt.h include if sanitizer_intercept_overriders.h sets SANITIZER_INTERCEPT_CRYPT_R to 0?

I guess I could keep the struct_crypt_data_sz declaration here and only omit its initialization in sanitizer_platform_limits_posix.cpp? Also a bit iffy, but maybe the best available option.

[thakis]

smaller

[thakis]

err!

[thakis]

Looks like pre-merge checks are now fairly happy with this. They come back red only due to clang-format complaints, and those look wrong (I left a comment about that on D100238).

compiler-rt/lib/sanitizer_common/sanitizer_platform_limits_posix.cpp

 #include <linux/if_plip.h>
 #include <linux/lp.h>
 #include <linux/mroute.h>
 #include <linux/mroute6.h>
 #include <linux/scc.h>
 #include <linux/serial.h>
 #include <sys/msg.h>
 #include <sys/ipc.h>
-#include <crypt.h>
 #endif  // SANITIZER_ANDROID
 
 #include <link.h>
 #include <sys/vfs.h>
 #include <sys/epoll.h>
 #include <linux/capability.h>
 #else
 #include <fstab.h>
 #endif // SANITIZER_LINUX
 
 #if SANITIZER_APPLE
 #include <net/ethernet.h>
 #include <sys/filio.h>
 #include <sys/sockio.h>
 #endif
 
 // Include these after system headers to avoid name clashes and ambiguities.
 #  include "sanitizer_common.h"
 #  include "sanitizer_internal_defs.h"
+#  include "sanitizer_platform_interceptors.h"
 #  include "sanitizer_platform_limits_posix.h"
 
+#if SANITIZER_INTERCEPT_CRYPT_R
+#include <crypt.h>
+#endif
+
 namespace __sanitizer {
   unsigned struct_utsname_sz = sizeof(struct utsname);
   unsigned struct_stat_sz = sizeof(struct stat);
 #if !SANITIZER_IOS && !(SANITIZER_APPLE && TARGET_CPU_ARM64)
   unsigned struct_stat64_sz = sizeof(struct stat64);
 #endif // !SANITIZER_IOS && !(SANITIZER_APPLE && TARGET_CPU_ARM64)
   unsigned struct_rusage_sz = sizeof(struct rusage);
   unsigned struct_tm_sz = sizeof(struct tm);
#    elif defined(__arm__) || defined(__i386__) || defined(__mips__) ||    \
         defined(__hexagon__)
 #      define SIZEOF_STRUCT_USTAT 20
 #    else
 #      error Unknown size of struct ustat
 #    endif
   unsigned struct_ustat_sz = SIZEOF_STRUCT_USTAT;
   unsigned struct_rlimit64_sz = sizeof(struct rlimit64);
   unsigned struct_statvfs64_sz = sizeof(struct statvfs64);
-  unsigned struct_crypt_data_sz = sizeof(struct crypt_data);
 #endif // SANITIZER_LINUX && !SANITIZER_ANDROID
 
+#if SANITIZER_INTERCEPT_CRYPT_R
+  unsigned struct_crypt_data_sz = sizeof(struct crypt_data);
+#endif
+
 #if SANITIZER_LINUX && !SANITIZER_ANDROID
   unsigned struct_timex_sz = sizeof(struct timex);
   unsigned struct_msqid_ds_sz = sizeof(struct msqid_ds);
   unsigned struct_mq_attr_sz = sizeof(struct mq_attr);
   unsigned struct_statvfs_sz = sizeof(struct statvfs);
 #endif // SANITIZER_LINUX && !SANITIZER_ANDROID
 
   const uptr sig_ign = (uptr)SIG_IGN;