diff options
author | Neil Horman <nhorman@openssl.org> | 2024-04-19 10:17:54 -0400 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2024-04-22 12:15:59 +0100 |
commit | 6ee369cd6ec751c03879da56178e75e2691e08cb (patch) | |
tree | 46c6de7e60b026d6b3d861de8d781a2b4119a858 /test | |
parent | fc9649f61a8ac5f980da6807214fcbbbae1c45aa (diff) |
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 <tomas@openssl.org>
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/24208)
Diffstat (limited to 'test')
-rw-r--r-- | test/prov_config_test.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/test/prov_config_test.c b/test/prov_config_test.c index fee2dffdb2..4f0cbc247b 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; |