The header uses `NULL` in both `CONFIG_KUNIT=y` and `=n` cases, but does not include it explicitly.
When `CONFIG_KUNIT=y`, the header is already getting included via the other headers, so it is not a problem for users.
However, when `CONFIG_KUNIT=n`, it is not, and thus a user could hit a build error when including `kunit/test-bug.h`, like we are doing later in this series [1].
Thus include `linux/stddef.h`, and do so outside the `#if`, since it is used in both cases.
Reported-by: Boqun Feng boqun.feng@gmail.com Closes: https://lore.kernel.org/rust-for-linux/ZJ8cNUW3oR2p+gL1@boqun-archlinux/ [1] Signed-off-by: Miguel Ojeda ojeda@kernel.org --- include/kunit/test-bug.h | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/include/kunit/test-bug.h b/include/kunit/test-bug.h index 30ca541b6ff2..47aa8f21ccce 100644 --- a/include/kunit/test-bug.h +++ b/include/kunit/test-bug.h @@ -9,6 +9,8 @@ #ifndef _KUNIT_TEST_BUG_H #define _KUNIT_TEST_BUG_H
+#include <linux/stddef.h> /* for NULL */ + #if IS_ENABLED(CONFIG_KUNIT)
#include <linux/jump_label.h> /* For static branch */