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

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

[Mark Brown]

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>
-- 
2.20.1

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

[Mark Brown]

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>
-- 
2.20.1


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH] kselftest/arm64: Add missing stddef.h include to BTI tests

[Catalin Marinas]

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

-- 
Catalin

Re: [PATCH] kselftest/arm64: Add missing stddef.h include to BTI tests

[Catalin Marinas]

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

-- 
Catalin


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel