Remove todos in decode_der2key.c and decode_ms2key.c
authorTomas Mraz <tomas@openssl.org>
Mon, 1 Mar 2021 15:14:30 +0000 (16:14 +0100)
committerPauli <ppzgs1@gmail.com>
Wed, 3 Mar 2021 00:00:21 +0000 (10:00 +1000)
Those TODOs do not really apply to 3.0 as the legacy internal
keys will stay.

Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14367)

providers/implementations/encode_decode/decode_der2key.c
providers/implementations/encode_decode/decode_ms2key.c

index fed30bf952041d39c45d7ff60da64cfaf1b8fff2..09601fc335e2a9dfb747fbf358b899f18876d339 100644 (file)
@@ -339,7 +339,7 @@ static int der2key_decode(void *vctx, OSSL_CORE_BIO *cin, int selection,
              * Tear out the low-level key pointer from the pkey,
              * but only if it matches the expected key type.
              *
-             * TODO: The check should be done with EVP_PKEY_is_a(), but
+             * The check should be done with EVP_PKEY_is_a(), but
              * as long as we still have #legacy internal keys, it's safer
              * to use the type numbers inside the provider.
              */
index 339b347fa025bc9d6de7a9e8d96cd6ca835c5310..e1741f4e537fa1d4adced9c8b18cb104008db137 100644 (file)
@@ -148,7 +148,7 @@ static int ms2key_post(struct ms2key_ctx_st *ctx, EVP_PKEY *pkey,
          * Tear out the low-level key pointer from the pkey,
          * but only if it matches the expected key type.
          *
-         * TODO(3.0): The check should be done with EVP_PKEY_is_a(), but
+         * The check should be done with EVP_PKEY_is_a(), but
          * as long as we still have #legacy internal keys, it's safer to
          * use the type numbers in side the provider.
          */