diff options
author | Pauli <pauli@openssl.org> | 2022-03-16 14:30:03 +1100 |
---|---|---|
committer | Pauli <pauli@openssl.org> | 2022-05-08 17:50:04 +1000 |
commit | 4e720f12fade8d433e5a0eb3ead9017193dac6e7 (patch) | |
tree | 5f76468fed6406d2f564f2485baa4edd351fd35b /crypto/evp | |
parent | 6d4f63469ed5a0f6e7ccde189e8a6d2bb1d2b26a (diff) |
These are both false positives since the `d` array is initialised by
the `DES_cfb_encrypt()` call via the `l2cn` macro. Rather than ignoring them
and having them crop up later, it's easier to just add an initialiser.
Reviewed-by: Matthias St. Pierre <Matthias.St.Pierre@ncp-e.com>
Reviewed-by: Dmitry Belyavskiy <beldmit@gmail.com>
(Merged from https://github.com/openssl/openssl/pull/17894)
Diffstat (limited to 'crypto/evp')
-rw-r--r-- | crypto/evp/e_des.c | 3 | ||||
-rw-r--r-- | crypto/evp/e_des3.c | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/crypto/evp/e_des.c b/crypto/evp/e_des.c index cd6e5af8d0..6eb49c0339 100644 --- a/crypto/evp/e_des.c +++ b/crypto/evp/e_des.c @@ -149,7 +149,8 @@ static int des_cfb1_cipher(EVP_CIPHER_CTX *ctx, unsigned char *out, const unsigned char *in, size_t inl) { size_t n, chunk = EVP_MAXCHUNK / 8; - unsigned char c[1], d[1]; + unsigned char c[1]; + unsigned char d[1] = { 0 }; /* Appease Coverity */ if (inl < chunk) chunk = inl; diff --git a/crypto/evp/e_des3.c b/crypto/evp/e_des3.c index 1e15918344..c0bc7fdd8f 100644 --- a/crypto/evp/e_des3.c +++ b/crypto/evp/e_des3.c @@ -165,7 +165,8 @@ static int des_ede3_cfb1_cipher(EVP_CIPHER_CTX *ctx, unsigned char *out, const unsigned char *in, size_t inl) { size_t n; - unsigned char c[1], d[1]; + unsigned char c[1]; + unsigned char d[1] = { 0 }; /* Appease Coverity */ if (!EVP_CIPHER_CTX_test_flags(ctx, EVP_CIPH_FLAG_LENGTH_BITS)) inl *= 8; |