diff options
| author | ndossche <niels.dossche@ugent.be> | 2023-02-02 16:11:16 +0100 |
|---|---|---|
| committer | Todd Short <todd.short@me.com> | 2023-02-08 10:34:49 -0500 |
| commit | abf654645dee168b229f3fa6a365f6a8e4dd7c31 (patch) | |
| tree | 626b3c7a55d5a5e77be36d02d3f70f1ed142d9d3 /crypto | |
| parent | 69b995c6fbc38163d69573803b7aa38ca64b074a (diff) | |
Fix incomplete error check on BIO_set_md()
BIO_set_md() can return an error value <= 0 according to my analysis
tool and the documentation. But only an error value == 0 is currently
checked. Fix it by changing the check condition.
CLA: trivial
Reviewed-by: Hugo Landau <hlandau@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
Reviewed-by: Todd Short <todd.short@me.com>
(Merged from https://github.com/openssl/openssl/pull/20195)
Diffstat (limited to 'crypto')
| -rw-r--r-- | crypto/cms/cms_lib.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/cms/cms_lib.c b/crypto/cms/cms_lib.c index a9e659b23d..b5a4b315a3 100644 --- a/crypto/cms/cms_lib.c +++ b/crypto/cms/cms_lib.c @@ -437,7 +437,7 @@ BIO *ossl_cms_DigestAlgorithm_init_bio(X509_ALGOR *digestAlgorithm, (void)ERR_pop_to_mark(); mdbio = BIO_new(BIO_f_md()); - if (mdbio == NULL || !BIO_set_md(mdbio, digest)) { + if (mdbio == NULL || BIO_set_md(mdbio, digest) <= 0) { ERR_raise(ERR_LIB_CMS, CMS_R_MD_BIO_INIT_ERROR); goto err; } |