kunit: Always print actual pointer values in asserts

KUnit assertions and expectations will print the values being tested. If
these are pointers (e.g., KUNIT_EXPECT_PTR_EQ(test, a, b)), these
pointers are currently printed with the %pK format specifier, which -- to
prevent information leaks which may compromise, e.g., ASLR -- are often
either hashed or replaced with ____ptrval____ or similar, making debugging
tests difficult.

By replacing %pK with %px as Documentation/core-api/printk-formats.rst
suggests, we disable this security feature for KUnit assertions and
expectations, allowing the actual pointer values to be printed. Given
that KUnit is not intended for use in production kernels, and the
pointers are only printed on failing tests, this seems like a worthwhile
tradeoff.

Signed-off-by: David Gow <davidgow@google.com>
Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>

diff --git a/lib/kunit/assert.c b/lib/kunit/assert.c
index b24bebca052dce7561ad366ab7f4e2a728be452a..02ecd0d7d80ab3582a7fea833e55aca591ccb0a0 100644 (file)
--- a/lib/kunit/assert.c
+++ b/lib/kunit/assert.c
@@ -118,10 +118,10 @@ void kunit_binary_ptr_assert_format(const struct kunit_assert *assert,
                         binary_assert->left_text,
                         binary_assert->operation,
                         binary_assert->right_text);
-       string_stream_add(stream, "\t\t%s == %pK\n",
+       string_stream_add(stream, "\t\t%s == %px\n",
                         binary_assert->left_text,
                         binary_assert->left_value);
-       string_stream_add(stream, "\t\t%s == %pK",
+       string_stream_add(stream, "\t\t%s == %px",
                         binary_assert->right_text,
                         binary_assert->right_value);
        kunit_assert_print_msg(assert, stream);