diff options
author | Shane Lontis <shane.lontis@oracle.com> | 2021-03-12 12:53:14 +1000 |
---|---|---|
committer | Pauli <ppzgs1@gmail.com> | 2021-03-14 15:35:43 +1000 |
commit | 8bfb8f34580cac3a53b0ac88dec566c19d6c8c53 (patch) | |
tree | 524d94b865a06d3b93811f614705814642c1afe4 /test | |
parent | cd3f8c1b11b0b9f4163bc8c62cbae38aec1b4030 (diff) |
Remove TODO in test/acvp_test.c related to setting AES-GCM iv.
Fixes #14330
Reviewed-by: Paul Dale <pauli@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/14525)
Diffstat (limited to 'test')
-rw-r--r-- | test/acvp_test.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/test/acvp_test.c b/test/acvp_test.c index 0be26f699b..cfed50b08d 100644 --- a/test/acvp_test.c +++ b/test/acvp_test.c @@ -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)) |