summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKlavishnik <evg.shtanov@gmail.com>2023-08-10 13:56:24 +0300
committerTomas Mraz <tomas@openssl.org>2023-10-10 17:15:54 +0200
commit8d120aef951d7bb7deac0b8b559f8003f5ea6384 (patch)
tree8a8a78f463960a2874781d605cdae2cac3f610c7
parentd0bf0106a5d25f4b9f13ff4e7187c4babe7694e1 (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.c3
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);