From 2828a2b92b89932d66b640e5047161448d522e2e Mon Sep 17 00:00:00 2001 From: Christian Heimes Date: Fri, 24 Feb 2017 08:20:15 +0100 Subject: [PATCH] C compilation fixes and hardening MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fix "implicit declaration of function ‘strlen’" in ipa_pwd_ntlm.c, credits to Lukas. Add -Werror=implicit-function-declaration to CFLAGS to point developers to missing includes. It causes compilation to fail when a developer forgets to add a required include. The problem is no longer hidden in a massive wall of text from make. Silence a harmless error from 389-DS slapi.h until the bug is fixed in downstream, https://pagure.io/389-ds-base/issue/48979 Signed-off-by: Christian Heimes Reviewed-By: Stanislav Laznicka --- configure.ac | 3 +++ util/ipa_pwd_ntlm.c | 1 + 2 files changed, 4 insertions(+) diff --git a/configure.ac b/configure.ac index 9ee281a16..af41f5e3a 100644 --- a/configure.ac +++ b/configure.ac @@ -24,6 +24,9 @@ LT_INIT AC_HEADER_STDC +dnl fail hard when includes statements are missing +CFLAGS+=" -Werror=implicit-function-declaration" + PKG_PROG_PKG_CONFIG AC_ARG_ENABLE([server], diff --git a/util/ipa_pwd_ntlm.c b/util/ipa_pwd_ntlm.c index f7099811e..263816dc5 100644 --- a/util/ipa_pwd_ntlm.c +++ b/util/ipa_pwd_ntlm.c @@ -26,6 +26,7 @@ #include #include +#include #include #include