summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authored <ed@FreeBSD.org>2014-09-01 20:18:09 +0000
committered <ed@FreeBSD.org>2014-09-01 20:18:09 +0000
commit6ddff9839c61d83a44ef8ac6a8592a077480cf0e (patch)
tree1ae0c8ff23ac962aec5916517d98c6c7c1c6f950
parent1812cd5a9f2e85cbf444f2b04bd4bc302bdb7b66 (diff)
downloadFreeBSD-src-6ddff9839c61d83a44ef8ac6a8592a077480cf0e.zip
FreeBSD-src-6ddff9839c61d83a44ef8ac6a8592a077480cf0e.tar.gz
Add -Wthread-safety to WARNS=6.
While there, add a NO_WTHREAD_SAFETY flag that can be used to disable this specific warning flag. Disable it for auditdistd. We can easily patch up auditdistd to have the right annotations to build, but as auditdistd is intended to be portable across other operating systems, it's not worth the effort. Approved by: brueffer@
-rw-r--r--share/mk/bsd.sys.mk3
-rw-r--r--usr.sbin/auditdistd/Makefile4
2 files changed, 7 insertions, 0 deletions
diff --git a/share/mk/bsd.sys.mk b/share/mk/bsd.sys.mk
index 58d6171..635d428 100644
--- a/share/mk/bsd.sys.mk
+++ b/share/mk/bsd.sys.mk
@@ -54,6 +54,9 @@ CWARNFLAGS+= -Wchar-subscripts -Winline -Wnested-externs -Wredundant-decls\
.if !defined(NO_WMISSING_VARIABLE_DECLARATIONS)
CWARNFLAGS.clang+= -Wmissing-variable-declarations
.endif
+.if !defined(NO_WTHREAD_SAFETY)
+CWARNFLAGS.clang+= -Wthread-safety
+.endif
.endif # WARNS >= 6
.if ${WARNS} >= 2 && ${WARNS} <= 4
# XXX Delete -Wuninitialized by default for now -- the compiler doesn't
diff --git a/usr.sbin/auditdistd/Makefile b/usr.sbin/auditdistd/Makefile
index ee18bca..b323dcd 100644
--- a/usr.sbin/auditdistd/Makefile
+++ b/usr.sbin/auditdistd/Makefile
@@ -30,4 +30,8 @@ YFLAGS+=-v
CLEANFILES=parse.c parse.h parse.output
+# auditdistd cannot use FreeBSD specific lock annotation macros. Disable
+# thread safety analysis completely.
+NO_WTHREAD_SAFETY=
+
.include <bsd.prog.mk>
OpenPOWER on IntegriCloud