summaryrefslogtreecommitdiffstats
path: root/lib/kunit
diff options
context:
space:
mode:
authorDavid Gow <davidgow@google.com>2019-11-21 15:50:58 -0800
committerShuah Khan <skhan@linuxfoundation.org>2020-03-25 12:12:33 -0600
commit2d68df6cc4bf5822d78cd0f067174d6e29a2f739 (patch)
treed5c2118f91d145cde8f5fd0fd08822a65c34915c /lib/kunit
parent0476e69f39377192d638c459d11400c6e9a6ffb0 (diff)
downloadlinux-2d68df6cc4bf5822d78cd0f067174d6e29a2f739.tar.gz
linux-2d68df6cc4bf5822d78cd0f067174d6e29a2f739.tar.bz2
linux-2d68df6cc4bf5822d78cd0f067174d6e29a2f739.zip
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>
Diffstat (limited to 'lib/kunit')
-rw-r--r--lib/kunit/assert.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/kunit/assert.c b/lib/kunit/assert.c
index b24bebca052d..02ecd0d7d80a 100644
--- 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);