diff options
| author | PW Hu <jlu.hpw@foxmail.com> | 2021-10-07 11:50:59 +0800 |
|---|---|---|
| committer | Tomas Mraz <tomas@openssl.org> | 2021-10-08 11:47:41 +0200 |
| commit | 4c09066ca62130c3a80365b1f94ade6c32b5d13b (patch) | |
| tree | c65e54a22aae4f03236621ee33c05ef867651ee9 /apps | |
| parent | 31ece182eea314ffe44dccb7d413ba16f29dc9ff (diff) | |
Fix unsafe BIO_get_md_ctx check
Reviewed-by: Matt Caswell <matt@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/16768)
(cherry picked from commit 59a3e7b29574ff45f62e825f6e9923f45060f142)
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/dgst.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/apps/dgst.c b/apps/dgst.c index 36a8c6fb08..bd23b76996 100644 --- a/apps/dgst.c +++ b/apps/dgst.c @@ -335,7 +335,7 @@ int dgst_main(int argc, char **argv) EVP_PKEY_CTX *pctx = NULL; int res; - if (!BIO_get_md_ctx(bmd, &mctx)) { + if (BIO_get_md_ctx(bmd, &mctx) <= 0) { BIO_printf(bio_err, "Error getting context\n"); goto end; } @@ -362,7 +362,7 @@ int dgst_main(int argc, char **argv) /* we use md as a filter, reading from 'in' */ else { EVP_MD_CTX *mctx = NULL; - if (!BIO_get_md_ctx(bmd, &mctx)) { + if (BIO_get_md_ctx(bmd, &mctx) <= 0) { BIO_printf(bio_err, "Error getting context\n"); goto end; } |