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:17:49 +0100 |
commit | 52f9e3a764ae7550b732c78f3831ab333cd8293b (patch) | |
tree | 01e0c3c492ee369b68707e247c6d2d91c097a557 | |
parent | 1fbbe1bb68d0585acf511547cbe03385e48e3822 (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)
(cherry picked from commit 6ee369cd6ec751c03879da56178e75e2691e08cb)
-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; |