diff options
author | Peiwei Hu <jlu.hpw@foxmail.com> | 2022-05-24 22:59:41 +0800 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2022-05-27 07:57:43 +0200 |
commit | f15e3f3aa95df743f0da793da952f87370efb4ff (patch) | |
tree | 16e2cf08982c06bc342b0eceb24c566b207132ac | |
parent | 8d9fec1781751d2106d899c6076eeb3da6930bfe (diff) |
Fix the incorrect checks of EVP_CIPHER_CTX_rand_key
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/18397)
-rw-r--r-- | apps/speed.c | 2 | ||||
-rw-r--r-- | test/evp_libctx_test.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/apps/speed.c b/apps/speed.c index 936107c5ef..c56f8e69f6 100644 --- a/apps/speed.c +++ b/apps/speed.c @@ -3602,7 +3602,7 @@ static void multiblock_speed(const EVP_CIPHER *evp_cipher, int lengths_single, goto err; } key = app_malloc(keylen, "evp_cipher key"); - if (!EVP_CIPHER_CTX_rand_key(ctx, key)) + if (EVP_CIPHER_CTX_rand_key(ctx, key) <= 0) app_bail_out("failed to generate random cipher key\n"); if (!EVP_EncryptInit_ex(ctx, NULL, NULL, key, NULL)) app_bail_out("failed to set cipher key\n"); diff --git a/test/evp_libctx_test.c b/test/evp_libctx_test.c index 65f5f694bf..2448c35a14 100644 --- a/test/evp_libctx_test.c +++ b/test/evp_libctx_test.c @@ -576,7 +576,7 @@ static int test_cipher_tdes_randkey(void) && TEST_int_ne(EVP_CIPHER_get_flags(tdes_cipher) & EVP_CIPH_RAND_KEY, 0) && TEST_ptr(ctx = EVP_CIPHER_CTX_new()) && TEST_true(EVP_CipherInit_ex(ctx, tdes_cipher, NULL, NULL, NULL, 1)) - && TEST_true(EVP_CIPHER_CTX_rand_key(ctx, key)); + && TEST_int_gt(EVP_CIPHER_CTX_rand_key(ctx, key), 0); EVP_CIPHER_CTX_free(ctx); EVP_CIPHER_free(tdes_cipher); |