diff options
author | Mickaël Salaün <mic@digikod.net> | 2024-05-11 19:14:41 +0200 |
---|---|---|
committer | Mickaël Salaün <mic@digikod.net> | 2024-05-11 19:18:45 +0200 |
commit | cc80aa9a22c00a5e23ea9b4933f9d3ec8f686cb2 (patch) | |
tree | dbcf1f0581f6b382981fa11259182201ef40689d | |
parent | selftests/landlock: Do not allocate memory in fixture data (diff) | |
download | linux-cc80aa9a22c00a5e23ea9b4933f9d3ec8f686cb2.tar.xz linux-cc80aa9a22c00a5e23ea9b4933f9d3ec8f686cb2.zip |
selftests/harness: Constify fixture variants
FIXTURE_VARIANT_ADD() types are passed as const pointers to
FIXTURE_TEARDOWN(). Make that explicit by constifying the variants
declarations.
Cc: Shuah Khan <skhan@linuxfoundation.org>
Cc: Will Drewry <wad@chromium.org>
Reviewed-by: Kees Cook <keescook@chromium.org>
Link: https://lore.kernel.org/r/20240511171445.904356-7-mic@digikod.net
Signed-off-by: Mickaël Salaün <mic@digikod.net>
-rw-r--r-- | tools/testing/selftests/kselftest_harness.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/testing/selftests/kselftest_harness.h b/tools/testing/selftests/kselftest_harness.h index 9d7178a71c2c..201040207c85 100644 --- a/tools/testing/selftests/kselftest_harness.h +++ b/tools/testing/selftests/kselftest_harness.h @@ -338,7 +338,7 @@ static inline pid_t clone3_vfork(void) * variant. */ #define FIXTURE_VARIANT_ADD(fixture_name, variant_name) \ - extern FIXTURE_VARIANT(fixture_name) \ + extern const FIXTURE_VARIANT(fixture_name) \ _##fixture_name##_##variant_name##_variant; \ static struct __fixture_variant_metadata \ _##fixture_name##_##variant_name##_object = \ @@ -350,7 +350,7 @@ static inline pid_t clone3_vfork(void) __register_fixture_variant(&_##fixture_name##_fixture_object, \ &_##fixture_name##_##variant_name##_object); \ } \ - FIXTURE_VARIANT(fixture_name) \ + const FIXTURE_VARIANT(fixture_name) \ _##fixture_name##_##variant_name##_variant = /** |