From e21980f4e5727d59a354bb43962647edbf13b817 Mon Sep 17 00:00:00 2001 From: Neil Horman Date: Fri, 5 Apr 2024 09:06:10 -0400 Subject: Fix up path generation to use OPENSSL_MODULES Reviewed-by: Matt Caswell Reviewed-by: Richard Levitte Reviewed-by: Tomas Mraz (Merged from https://github.com/openssl/openssl/pull/24025) (cherry picked from commit 4e3c1e6206251c59855362d6d2edab4621c31dec) --- crypto/provider_core.c | 9 +++++---- test/prov_config_test.c | 15 ++++++++++++++- 2 files changed, 19 insertions(+), 5 deletions(-) diff --git a/crypto/provider_core.c b/crypto/provider_core.c index 4bbb928006..297b281a39 100644 --- a/crypto/provider_core.c +++ b/crypto/provider_core.c @@ -573,10 +573,6 @@ OSSL_PROVIDER *ossl_provider_new(OSSL_LIB_CTX *libctx, const char *name, /* provider_new() generates an error, so no need here */ prov = provider_new(name, template.init, template.parameters); - if (!ossl_provider_set_module_path(prov, template.path)) { - ossl_provider_free(prov); - return NULL; - } if (params != NULL) /* We copied the parameters, let's free them */ sk_INFOPAIR_pop_free(template.parameters, infopair_free); @@ -584,6 +580,11 @@ OSSL_PROVIDER *ossl_provider_new(OSSL_LIB_CTX *libctx, const char *name, if (prov == NULL) return NULL; + if (!ossl_provider_set_module_path(prov, template.path)) { + ossl_provider_free(prov); + return NULL; + } + prov->libctx = libctx; #ifndef FIPS_MODULE prov->error_lib = ERR_get_next_error_library(); diff --git a/test/prov_config_test.c b/test/prov_config_test.c index 2fac741a3d..fee2dffdb2 100644 --- a/test/prov_config_test.c +++ b/test/prov_config_test.c @@ -72,14 +72,27 @@ static int test_recursive_config(void) return testresult; } +#define P_TEST_PATH "/../test/p_test.so" static int test_path_config(void) { OSSL_LIB_CTX *ctx = NULL; OSSL_PROVIDER *prov; int testresult = 0; struct stat sbuf; + char *module_path = getenv("OPENSSL_MODULES"); + char *full_path = NULL; + int rc; - if (stat("../test/p_test.so", &sbuf) == -1) + full_path = OPENSSL_zalloc(strlen(module_path) + strlen(P_TEST_PATH) + 1); + if (!TEST_ptr(full_path)) + return 0; + + strcpy(full_path, module_path); + full_path = strcat(full_path, P_TEST_PATH); + TEST_info("full path is %s", full_path); + rc = stat(full_path, &sbuf); + OPENSSL_free(full_path); + if (rc == -1) return TEST_skip("Skipping modulepath test as provider not present"); if (!TEST_ptr(pathedconfig)) -- cgit v1.2.3