Explicitly include stddef.h when building the BTI tests so that we have a definition of NULL, with at least some toolchains this is not done implicitly by anything else:
test.c: In function ‘start’: test.c:214:25: error: ‘NULL’ undeclared (first use in this function) 214 | sigaction(SIGILL, &sa, NULL); | ^~~~ test.c:20:1: note: ‘NULL’ is defined in header ‘<stddef.h>’; did you forget to ‘#include <stddef.h>’?
Signed-off-by: Mark Brown broonie@kernel.org --- tools/testing/selftests/arm64/bti/test.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/tools/testing/selftests/arm64/bti/test.c b/tools/testing/selftests/arm64/bti/test.c index 656b04976ccc..67b77ab83c20 100644 --- a/tools/testing/selftests/arm64/bti/test.c +++ b/tools/testing/selftests/arm64/bti/test.c @@ -6,6 +6,7 @@
#include "system.h"
+#include <stddef.h> #include <linux/errno.h> #include <linux/auxvec.h> #include <linux/signal.h>
On Fri, 7 May 2021 17:25:42 +0100, Mark Brown wrote:
Explicitly include stddef.h when building the BTI tests so that we have a definition of NULL, with at least some toolchains this is not done implicitly by anything else:
test.c: In function ‘start’: test.c:214:25: error: ‘NULL’ undeclared (first use in this function) 214 | sigaction(SIGILL, &sa, NULL); | ^~~~ test.c:20:1: note: ‘NULL’ is defined in header ‘<stddef.h>’; did you forget to ‘#include <stddef.h>’?
Applied to arm64 (for-next/fixes), thanks!
[1/1] kselftest/arm64: Add missing stddef.h include to BTI tests https://git.kernel.org/arm64/c/a1bed090fc56
linux-kselftest-mirror@lists.linaro.org