diff mbox series

kselftest/arm64: Add missing stddef.h include to BTI tests

Message ID 20210507162542.23149-1-broonie@kernel.org (mailing list archive)
State Accepted
Commit a1bed090fc56e6e24517d96bc076595544fb5317
Headers show
Series kselftest/arm64: Add missing stddef.h include to BTI tests | expand

Commit Message

Mark Brown May 7, 2021, 4:25 p.m. UTC
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(+)

Comments

Catalin Marinas May 10, 2021, 12:56 p.m. UTC | #1
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
diff mbox series

Patch

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>