Fix sendmmsg and recvmmsg rtsan interceptor for MUSL

[kongy]

MUSL have different signatures for sendmmsg and recvmmsg.

This fixes build breakage from #123484.

[llvmbot]

@llvm/pr-subscribers-compiler-rt-sanitizer

Author: Yi Kong (kongy)

Changes

MUSL have different signatures for sendmmsg and recvmmsg.

---

Full diff: https://github.com/llvm/llvm-project/pull/123907.diff

1 Files Affected:

• (modified) compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp (+9-1)

diff --git a/compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp b/compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp
index 112191f52648e7..008cc676a6ae55 100644
--- a/compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp
+++ b/compiler-rt/lib/rtsan/rtsan_interceptors_posix.cpp
@@ -894,8 +894,13 @@ INTERCEPTOR(ssize_t, sendmsg, int socket, const struct msghdr *message,
 }
 
 #if SANITIZER_INTERCEPT_SENDMMSG
+#if SANITIZER_MUSL
+INTERCEPTOR(int, sendmmsg, int socket, struct mmsghdr *message,
+            unsigned int len, unsigned int flags) {
+#else
 INTERCEPTOR(int, sendmmsg, int socket, struct mmsghdr *message,
             unsigned int len, int flags) {
+#endif
   __rtsan_notify_intercepted_call("sendmmsg");
   return REAL(sendmmsg)(socket, message, len, flags);
 }
@@ -927,7 +932,10 @@ INTERCEPTOR(ssize_t, recvmsg, int socket, struct msghdr *message, int flags) {
 }
 
 #if SANITIZER_INTERCEPT_RECVMMSG
-#if defined(__GLIBC_MINOR__) && __GLIBC_MINOR__ < 21
+#if SANITIZER_MUSL
+INTERCEPTOR(int, recvmmsg, int socket, struct mmsghdr *message,
+            unsigned int len, unsigned int flags, struct timespec *timeout) {
+#elif defined(__GLIBC_MINOR__) && __GLIBC_MINOR__ < 21
 INTERCEPTOR(int, recvmmsg, int socket, struct mmsghdr *message,
             unsigned int len, int flags, const struct timespec *timeout) {
 #else

[devnexen]

What a mess :) thanks for fixing it.

[devnexen]

LGTM but waiting @cjappl

[cjappl]

LGTM thank you!!

This is just musings doesn't affect this review - I wonder if there is a better way to organize all the differences of signatures across versions/libcs/OSs. One that is more readable or maintainable.

As we continue adding interceptors we may want to re-organize in some way

[appujee]

Fixes #123689