Commit f32f8682ea for openssl.org
commit f32f8682eac94360684d9856b7f72de4948b8637
Author: Nikola Pajkovsky <nikolap@openssl.org>
Date: Tue Jun 16 08:32:35 2026 +0200
providers/fips/fipsprov.c, test/p_test.c: remove c_gettable_params static global
c_gettable_params is never read anywhere in the files; it was dead
storage. Remove it.
Signed-off-by: Nikola Pajkovsky <nikolap@openssl.org>
Reviewed-by: Norbert Pocs <norbertp@openssl.org>
Reviewed-by: Neil Horman <nhorman@openssl.org>
Reviewed-by: Eugene Syromiatnikov <esyr@openssl.org>
MergeDate: Sun Jun 28 17:51:55 2026
(Merged from https://github.com/openssl/openssl/pull/31508)
diff --git a/providers/fips/fipsprov.c b/providers/fips/fipsprov.c
index 3292dce334..5f331920ba 100644
--- a/providers/fips/fipsprov.c
+++ b/providers/fips/fipsprov.c
@@ -63,7 +63,6 @@ extern OSSL_FUNC_core_thread_start_fn *c_thread_start;
*/
/* Functions provided by the core */
-static OSSL_FUNC_core_gettable_params_fn *c_gettable_params;
static OSSL_FUNC_core_get_params_fn *c_get_params;
OSSL_FUNC_core_thread_start_fn *c_thread_start;
static OSSL_FUNC_core_new_error_fn *c_new_error;
@@ -840,9 +839,6 @@ int OSSL_provider_init_int(const OSSL_CORE_HANDLE *handle,
case OSSL_FUNC_CORE_GET_LIBCTX:
set_func(c_get_libctx, OSSL_FUNC_core_get_libctx(in));
break;
- case OSSL_FUNC_CORE_GETTABLE_PARAMS:
- set_func(c_gettable_params, OSSL_FUNC_core_gettable_params(in));
- break;
case OSSL_FUNC_CORE_GET_PARAMS:
set_func(c_get_params, OSSL_FUNC_core_get_params(in));
break;
diff --git a/test/p_test.c b/test/p_test.c
index 1e38bee717..655cb7ce6f 100644
--- a/test/p_test.c
+++ b/test/p_test.c
@@ -43,7 +43,6 @@ typedef struct p_test_ctx {
OSSL_LIB_CTX *libctx;
} P_TEST_CTX;
-static OSSL_FUNC_core_gettable_params_fn *c_gettable_params = NULL;
static OSSL_FUNC_core_get_params_fn *c_get_params = NULL;
static OSSL_FUNC_core_new_error_fn *c_new_error;
static OSSL_FUNC_core_set_error_debug_fn *c_set_error_debug;
@@ -258,9 +257,6 @@ int OSSL_provider_init(const OSSL_CORE_HANDLE *handle,
for (; in->function_id != 0; in++) {
switch (in->function_id) {
- case OSSL_FUNC_CORE_GETTABLE_PARAMS:
- c_gettable_params = OSSL_FUNC_core_gettable_params(in);
- break;
case OSSL_FUNC_CORE_GET_PARAMS:
c_get_params = OSSL_FUNC_core_get_params(in);
break;