From 79cabd7e277ccb0763bd2f6438abcb089dcbdff1 Mon Sep 17 00:00:00 2001 From: Pauli Date: Mon, 31 May 2021 14:29:34 +1000 Subject: [PATCH] rsa: remove TODOs Reviewed-by: Tim Hudson (Merged from https://github.com/openssl/openssl/pull/15539) --- crypto/rsa/rsa_backend.c | 1 - crypto/rsa/rsa_ossl.c | 10 ---------- 2 files changed, 11 deletions(-) diff --git a/crypto/rsa/rsa_backend.c b/crypto/rsa/rsa_backend.c index 5b7d60d6e10..e824dcaf3cd 100644 --- a/crypto/rsa/rsa_backend.c +++ b/crypto/rsa/rsa_backend.c @@ -270,7 +270,6 @@ int ossl_rsa_pss_params_30_fromdata(RSA_PSS_PARAMS_30 *pss_params, else if (!OSSL_PARAM_get_utf8_ptr(param_mgf, &mgfname)) return 0; - /* TODO Revisit this if / when a new MGF algorithm appears */ if (strcasecmp(param_mgf->data, ossl_rsa_mgf_nid2name(default_maskgenalg_nid)) != 0) return 0; diff --git a/crypto/rsa/rsa_ossl.c b/crypto/rsa/rsa_ossl.c index 1817392e762..c417a4b8f6d 100644 --- a/crypto/rsa/rsa_ossl.c +++ b/crypto/rsa/rsa_ossl.c @@ -780,16 +780,6 @@ static int rsa_ossl_mod_exp(BIGNUM *r0, const BIGNUM *I, RSA *rsa, BN_CTX *ctx) } #ifndef FIPS_MODULE - /* - * calculate m_i in multi-prime case - * - * TODO: - * 1. squash the following two loops and calculate |m_i| there. - * 2. remove cc and reuse |c|. - * 3. remove |dmq1| and |dmp1| in previous block and use |di|. - * - * If these things are done, the code will be more readable. - */ if (ex_primes > 0) { BIGNUM *di = BN_new(), *cc = BN_new(); -- 2.47.3