From: Zichen Xie <[email protected]>

kunit_kzalloc() may return a NULL pointer, dereferencing it without
NULL check may lead to NULL dereference.
Add a NULL check for test_state.

Fixes: d03c720e03bd ("kunit: Add APIs for managing devices")
Signed-off-by: Zichen Xie <[email protected]>
---
 lib/kunit/kunit-test.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/kunit/kunit-test.c b/lib/kunit/kunit-test.c
index 37e02be1e710..d9c781c859fd 100644
--- a/lib/kunit/kunit-test.c
+++ b/lib/kunit/kunit-test.c
@@ -805,6 +805,8 @@ static void kunit_device_driver_test(struct kunit *test)
        struct device *test_device;
        struct driver_test_state *test_state = kunit_kzalloc(test, 
sizeof(*test_state), GFP_KERNEL);
 
+       KUNIT_ASSERT_NOT_ERR_OR_NULL(test, test_state);
+
        test->priv = test_state;
        test_driver = kunit_driver_create(test, "my_driver");
 
-- 
2.34.1


Reply via email to