aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorAndreas Schneider <asn@samba.org>2018-09-28 11:01:15 +0200
committerAndreas Schneider <asn@samba.org>2018-10-23 15:38:31 +0200
commita5df8fc8f5f22ba561a39b3eb752766cb62d7f71 (patch)
tree6bafb04c04d4886ce4c494b4ffad8cc69d4673df /src
parentda4b908371ddc05a8402794b6929979449e4132a (diff)
downloadsocket_wrapper-a5df8fc8f5f22ba561a39b3eb752766cb62d7f71.tar.gz
socket_wrapper-a5df8fc8f5f22ba561a39b3eb752766cb62d7f71.tar.xz
socket_wrapper-a5df8fc8f5f22ba561a39b3eb752766cb62d7f71.zip
cmake: Suppress helgrind race condition warning we created on purpose
Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org>
Diffstat (limited to 'src')
-rw-r--r--src/socket_wrapper.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/socket_wrapper.c b/src/socket_wrapper.c
index 36482c6..7a27b40 100644
--- a/src/socket_wrapper.c
+++ b/src/socket_wrapper.c
@@ -659,6 +659,12 @@ static void swrap_mutex_unlock(pthread_mutex_t *mutex)
}
}
+/*
+ * These macros have a thread race condition on purpose!
+ *
+ * This is an optimization to avoid locking each time we check if the symbol is
+ * bound.
+ */
#define swrap_bind_symbol_libc(sym_name) \
if (swrap.libc.symbols._libc_##sym_name.obj == NULL) { \
swrap_mutex_lock(&libc_symbol_binding_mutex); \