diff options
| author | Miguel Ojeda <ojeda@kernel.org> | 2022-05-02 11:36:25 +0200 |
|---|---|---|
| committer | Shuah Khan <skhan@linuxfoundation.org> | 2022-05-16 13:23:00 -0600 |
| commit | 7466886b400b1904ce30fa311904849e314a2cf4 (patch) | |
| tree | 8507f2f6ad0365f187344d491e6e5417857944ac /lib/kunit | |
| parent | 0453f984a7b9458f0e469afb039f2841308b1bef (diff) | |
kunit: take `kunit_assert` as `const`
The `kunit_do_failed_assertion` function passes its
`struct kunit_assert` argument to `kunit_fail`. This one,
in turn, calls its `format` field passing the assert again
as a `const` pointer.
Therefore, the whole chain may be made `const`.
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Reviewed-by: Daniel Latypov <dlatypov@google.com>
Reviewed-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'lib/kunit')
| -rw-r--r-- | lib/kunit/test.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/kunit/test.c b/lib/kunit/test.c index 65c56bd0545d..a5053a07409f 100644 --- a/lib/kunit/test.c +++ b/lib/kunit/test.c @@ -244,7 +244,7 @@ static void kunit_print_string_stream(struct kunit *test, } static void kunit_fail(struct kunit *test, const struct kunit_loc *loc, - enum kunit_assert_type type, struct kunit_assert *assert, + enum kunit_assert_type type, const struct kunit_assert *assert, const struct va_format *message) { struct string_stream *stream; @@ -284,7 +284,7 @@ static void __noreturn kunit_abort(struct kunit *test) void kunit_do_failed_assertion(struct kunit *test, const struct kunit_loc *loc, enum kunit_assert_type type, - struct kunit_assert *assert, + const struct kunit_assert *assert, const char *fmt, ...) { va_list args; |