mirrors/openssl
0
0
Fork 0
mirror of https://github.com/openssl/openssl.git synced 2024-12-12 19:43:42 +08:00
Code Issues Packages Projects Releases Wiki Activity

Fix coverity 1516093 tainted scalar

Browse Source 
|uclen| is created from three byte values, so this seems a bit
redundant, but if it makes coverity happy

Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/19519)
This commit is contained in:
Todd Short 2022-10-27 10:27:29 -04:00 committed by Tomas Mraz
parent 943051d0f9
commit 00e38edcfb
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ssl/t1_trce.c
View File

@ -1345,7 +1345,8 @@ static int ssl_print_compressed_certificates(BIO *bio, const SSL_CONNECTION *sc,
if (!ossl_comp_has_alg(alg))
return 0;
if (uclen == 0 || (ucdata = OPENSSL_malloc(uclen)) == NULL)
/* Check against certificate maximum size (coverity) */
if (uclen == 0 || uclen > 0xFFFFFF || (ucdata = OPENSSL_malloc(uclen)) == NULL)
return 0;
switch (alg) {