Gitweb:
https://git.exim.org/exim.git/commitdiff/780ea2a5cc313ee12c0ad7823b03feae17c3b08b
Commit: 780ea2a5cc313ee12c0ad7823b03feae17c3b08b
Parent: 8b78698fadd197d75137df1bb0b0aae4f601f7f3
Author: Jeremy Harris <jgh146exb@???>
AuthorDate: Mon Sep 20 22:41:04 2021 +0100
Committer: Jeremy Harris <jgh146exb@???>
CommitDate: Mon Sep 20 22:41:04 2021 +0100
OpenBSD: disable compiler-time param checking for string_sprintf() etc
---
src/src/mytypes.h | 15 +++++++++++----
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/src/src/mytypes.h b/src/src/mytypes.h
index 8ae7cbb..38ee93d 100644
--- a/src/src/mytypes.h
+++ b/src/src/mytypes.h
@@ -30,17 +30,20 @@ local_scan.h includes it and exim.h includes them both (to get this earlier). */
/* If gcc is being used to compile Exim, we can use its facility for checking
-the arguments of printf-like functions. This is done by a macro. */
+the arguments of printf-like functions. This is done by a macro.
+OpenBSD has unfortunately taken to objecting to use of %n in printf
+so we have to give up on all of the available parameter checking. */
#if defined(__GNUC__) || defined(__clang__)
-# define PRINTF_FUNCTION(A,B) __attribute__((format(printf,A,B)))
+# ifndef __OpenBSD__
+# define PRINTF_FUNCTION(A,B) __attribute__((format(printf,A,B)))
+# endif
# define ARG_UNUSED __attribute__((__unused__))
# define WARN_UNUSED_RESULT __attribute__((__warn_unused_result__))
# define ALLOC __attribute__((malloc))
# define ALLOC_SIZE(A) __attribute__((alloc_size(A)))
# define NORETURN __attribute__((noreturn))
#else
-# define PRINTF_FUNCTION(A,B)
# define ARG_UNUSED /**/
# define WARN_UNUSED_RESULT /**/
# define ALLOC /**/
@@ -48,10 +51,14 @@ the arguments of printf-like functions. This is done by a macro. */
# define NORETURN /**/
#endif
+#ifndef PRINTF_FUNCTION
+# define PRINTF_FUNCTION(A,B) /**/
+#endif
+
#ifdef WANT_DEEPER_PRINTF_CHECKS
# define ALMOST_PRINTF(A, B) PRINTF_FUNCTION(A, B)
#else
-# define ALMOST_PRINTF(A, B)
+# define ALMOST_PRINTF(A, B) /**/
#endif
