From 6ee369cd6ec751c03879da56178e75e2691e08cb Mon Sep 17 00:00:00 2001 From: Neil Horman Date: Fri, 19 Apr 2024 10:17:54 -0400 Subject: [PATCH] Fix missing NULL check in prov_config_test coverity-1596500 caught a missing null check. We should never hit it as the test harness always sets the environment variable, but lets add the check for safety Reviewed-by: Tomas Mraz Reviewed-by: Tom Cosgrove Reviewed-by: Matt Caswell (Merged from https://github.com/openssl/openssl/pull/24208) --- test/prov_config_test.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/test/prov_config_test.c b/test/prov_config_test.c index fee2dffdb20..4f0cbc247b4 100644 --- a/test/prov_config_test.c +++ b/test/prov_config_test.c @@ -83,6 +83,9 @@ static int test_path_config(void) char *full_path = NULL; int rc; + if (!TEST_ptr(module_path)) + return 0; + full_path = OPENSSL_zalloc(strlen(module_path) + strlen(P_TEST_PATH) + 1); if (!TEST_ptr(full_path)) return 0; -- 2.47.2