From 99b9aa95c435058bd35fb3b83b885894e2247c82 Mon Sep 17 00:00:00 2001 From: Pauli Date: Tue, 11 Feb 2020 09:13:33 +1000 Subject: [PATCH] Remove unused OSSL_PARAM_construct_from_text() function. This function is recently introduced and never called by the library or tests. Reviewed-by: Matt Caswell (Merged from https://github.com/openssl/openssl/pull/11053) --- crypto/params_from_text.c | 31 ------------------- crypto/sm2/sm2_pmeth.c | 3 +- ....pod => OSSL_PARAM_allocate_from_text.pod} | 15 ++------- include/openssl/params.h | 5 --- util/libcrypto.num | 1 - 5 files changed, 4 insertions(+), 51 deletions(-) rename doc/man3/{OSSL_PARAM_construct_from_text.pod => OSSL_PARAM_allocate_from_text.pod} (86%) diff --git a/crypto/params_from_text.c b/crypto/params_from_text.c index 053b93d2c38..59cee5f1159 100644 --- a/crypto/params_from_text.c +++ b/crypto/params_from_text.c @@ -160,37 +160,6 @@ static int construct_from_text(OSSL_PARAM *to, const OSSL_PARAM *paramdef, return 1; } -int OSSL_PARAM_construct_from_text(OSSL_PARAM *to, - const OSSL_PARAM *paramdefs, - const char *key, const char *value, - size_t value_n, - void *buf, size_t *buf_n) -{ - const OSSL_PARAM *paramdef = NULL; - int ishex = 0; - BIGNUM *tmpbn = NULL; - int ok = 0; - - if (to == NULL || paramdefs == NULL) - return 0; - - if (!prepare_from_text(paramdefs, key, value, value_n, - ¶mdef, &ishex, buf_n, &tmpbn)) - return 0; - - /* - * The user gets the expected buffer size back even if the buffer isn't - * allocated. - */ - if (buf == NULL) - return 1; - - ok = construct_from_text(to, paramdef, value, value_n, ishex, - buf, *buf_n, tmpbn); - BN_free(tmpbn); - return ok; -} - int OSSL_PARAM_allocate_from_text(OSSL_PARAM *to, const OSSL_PARAM *paramdefs, const char *key, const char *value, diff --git a/crypto/sm2/sm2_pmeth.c b/crypto/sm2/sm2_pmeth.c index 681a0ab130a..c3ba9280c59 100644 --- a/crypto/sm2/sm2_pmeth.c +++ b/crypto/sm2/sm2_pmeth.c @@ -253,8 +253,7 @@ static int pkey_sm2_ctrl_str(EVP_PKEY_CTX *ctx, } else if (strcmp(type, "sm2_hex_id") == 0) { /* * TODO(3.0): reconsider the name "sm2_hex_id", OR change - * OSSL_PARAM_construct_from_text() / OSSL_PARAM_allocate_from_text() - * to handle infix "_hex_" + * OSSL_PARAM_allocate_from_text() to handle infix "_hex_" */ hex_id = OPENSSL_hexstr2buf((const char *)value, &hex_len); if (hex_id == NULL) { diff --git a/doc/man3/OSSL_PARAM_construct_from_text.pod b/doc/man3/OSSL_PARAM_allocate_from_text.pod similarity index 86% rename from doc/man3/OSSL_PARAM_construct_from_text.pod rename to doc/man3/OSSL_PARAM_allocate_from_text.pod index 7e6c3e91939..c16491e702e 100644 --- a/doc/man3/OSSL_PARAM_construct_from_text.pod +++ b/doc/man3/OSSL_PARAM_allocate_from_text.pod @@ -2,18 +2,13 @@ =head1 NAME -OSSL_PARAM_construct_from_text, OSSL_PARAM_allocate_from_text +OSSL_PARAM_allocate_from_text - OSSL_PARAM construction utilities =head1 SYNOPSIS #include - int OSSL_PARAM_construct_from_text(OSSL_PARAM *to, - const OSSL_PARAM *paramdefs, - const char *key, const char *value, - size_t value_n, - void *buf, size_t *buf_n) int OSSL_PARAM_allocate_from_text(OSSL_PARAM *to, const OSSL_PARAM *paramdefs, const char *key, const char *value, @@ -30,7 +25,7 @@ OpenSSL 3.0 introduces a new mechanism to do the same thing with an array of parameters that contain name, value, value type and value size (see L for more information). -OSSL_PARAM_construct_from_text() takes a control I, I and +OSSL_PARAM_allocate_from_text() takes a control I, I and value size I, and given a parameter descriptor array I, it converts the value to something suitable for L and stores that in the buffer I, and modifies @@ -42,9 +37,6 @@ left untouched, allowing a caller to find out how large the buffer should be. I needs to be correctly aligned for the type of the B I. - -OSSL_PARAM_allocate_from_text() works like OSSL_PARAM_construct_from_text(), -except it allocates the buffer internally. The caller must remember to free the data of I when it's not useful any more. @@ -57,8 +49,7 @@ as parameter value. =head1 RETURN VALUES -OSSL_PARAM_construct_from_text() and OSSL_PARAM_allocate_from_text() -returns 1 on success, and 0 on error. +OSSL_PARAM_allocate_from_text() returns 1 on success, and 0 on error. =head1 NOTES diff --git a/include/openssl/params.h b/include/openssl/params.h index aec3bc18b80..a5d2fd4f41b 100644 --- a/include/openssl/params.h +++ b/include/openssl/params.h @@ -89,11 +89,6 @@ OSSL_PARAM OSSL_PARAM_construct_octet_ptr(const char *key, void **buf, size_t bsize); OSSL_PARAM OSSL_PARAM_construct_end(void); -int OSSL_PARAM_construct_from_text(OSSL_PARAM *to, - const OSSL_PARAM *paramdefs, - const char *key, const char *value, - size_t value_n, - void *buf, size_t *buf_n); int OSSL_PARAM_allocate_from_text(OSSL_PARAM *to, const OSSL_PARAM *paramdefs, const char *key, const char *value, diff --git a/util/libcrypto.num b/util/libcrypto.num index 29e37b4a41e..a87630b6333 100644 --- a/util/libcrypto.num +++ b/util/libcrypto.num @@ -4688,7 +4688,6 @@ EC_KEY_new_ex ? 3_0_0 EXIST::FUNCTION:EC EC_KEY_new_by_curve_name_ex ? 3_0_0 EXIST::FUNCTION:EC OPENSSL_hexstr2buf_ex ? 3_0_0 EXIST::FUNCTION: OPENSSL_buf2hexstr_ex ? 3_0_0 EXIST::FUNCTION: -OSSL_PARAM_construct_from_text ? 3_0_0 EXIST::FUNCTION: OSSL_PARAM_allocate_from_text ? 3_0_0 EXIST::FUNCTION: EVP_MD_gettable_params ? 3_0_0 EXIST::FUNCTION: EVP_MD_CTX_settable_params ? 3_0_0 EXIST::FUNCTION: -- 2.47.2