Remove TODO in test/acvp_test.c related to setting AES-GCM iv.
authorShane Lontis <shane.lontis@oracle.com>
Fri, 12 Mar 2021 02:53:14 +0000 (12:53 +1000)
committerPauli <ppzgs1@gmail.com>
Sun, 14 Mar 2021 05:35:43 +0000 (15:35 +1000)
Fixes #14330

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

test/acvp_test.c

index 0be26f699b4ffba034d1be4d64c8a0f8d643fb6a..cfed50b08d9d016231b8ae291df6789c46f04372 100644 (file)
@@ -837,8 +837,9 @@ static int aes_gcm_enc_dec(const char *alg,
             goto err;
     }
     /*
-     * TODO(3.0): The IV should not be set outside the boundary as it is now.
-     * It needs to be fed in via a dummy entropy source for this test.
+     * For testing purposes the IV it being set here. In a compliant application
+     * the IV would be generated internally. A fake entropy source could also
+     * be used to feed in the random IV bytes (see fake_random.c)
      */
     if (!TEST_true(EVP_CipherInit_ex(ctx, NULL, NULL, key, iv, enc))
         || !TEST_true(EVP_CIPHER_CTX_set_padding(ctx, 0))