diff options
| author | PW Hu <jlu.hpw@foxmail.com> | 2021-11-01 15:08:51 +0800 |
|---|---|---|
| committer | PW Hu <jlu.hpw@foxmail.com> | 2021-11-01 15:08:51 +0800 |
| commit | 944fcfc69d16dfd20decdd9cd105436f0043dbe0 (patch) | |
| tree | afd998d08e5e7b4b6a98419dcada17e4fb4d39a3 /crypto/evp/ctrl_params_translate.c | |
| parent | a87c3247ca641f2593391bf44d47e3dccc7f8d73 (diff) | |
| download | openssl-944fcfc69d16dfd20decdd9cd105436f0043dbe0.tar.gz | |
Fix incorrect return check of BN_bn2nativepad
Reviewed-by: Kurt Roeckx <kurt@roeckx.be>
Reviewed-by: Paul Dale <pauli@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/16943)
Diffstat (limited to 'crypto/evp/ctrl_params_translate.c')
| -rw-r--r-- | crypto/evp/ctrl_params_translate.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/crypto/evp/ctrl_params_translate.c b/crypto/evp/ctrl_params_translate.c index b17ce3cbf9..d17017a78e 100644 --- a/crypto/evp/ctrl_params_translate.c +++ b/crypto/evp/ctrl_params_translate.c @@ -465,8 +465,8 @@ static int default_fixup_args(enum state state, ERR_raise(ERR_LIB_EVP, ERR_R_MALLOC_FAILURE); return 0; } - if (!BN_bn2nativepad(ctx->p2, - ctx->allocated_buf, ctx->buflen)) { + if (BN_bn2nativepad(ctx->p2, + ctx->allocated_buf, ctx->buflen) < 0) { OPENSSL_free(ctx->allocated_buf); ctx->allocated_buf = NULL; return 0; |