diff options
author | Klavishnik <evg.shtanov@gmail.com> | 2023-08-10 13:56:24 +0300 |
---|---|---|
committer | Tomas Mraz <tomas@openssl.org> | 2023-10-10 17:15:54 +0200 |
commit | 8d120aef951d7bb7deac0b8b559f8003f5ea6384 (patch) | |
tree | 8a8a78f463960a2874781d605cdae2cac3f610c7 | |
parent | d0bf0106a5d25f4b9f13ff4e7187c4babe7694e1 (diff) |
Added check for the return value of the RAND_bytes() function
Call app_bail_out if RAND_bytes() fails.
Also changed the output parameter of RAND_bytes() to inp as
writing to encrypted output buffer does not make sense.
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tom Cosgrove <tom.cosgrove@arm.com>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/21706)
-rw-r--r-- | apps/speed.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/apps/speed.c b/apps/speed.c index 88d389523b..44e3af7591 100644 --- a/apps/speed.c +++ b/apps/speed.c @@ -4747,7 +4747,8 @@ static void multiblock_speed(const EVP_CIPHER *evp_cipher, int lengths_single, } else { int pad; - RAND_bytes(out, 16); + if (RAND_bytes(inp, 16) <= 0) + app_bail_out("error setting random bytes\n"); len += 16; aad[11] = (unsigned char)(len >> 8); aad[12] = (unsigned char)(len); |