diff options
| author | Richard Levitte <levitte@openssl.org> | 2020-01-12 01:05:01 +0100 |
|---|---|---|
| committer | Richard Levitte <levitte@openssl.org> | 2020-01-15 01:17:14 +0100 |
| commit | 76123661a1db136b9ef368dc296a628818e7a4cc (patch) | |
| tree | 7775b37879a1b290976bd10ada14b9dde5590497 /crypto/evp/m_sigver.c | |
| parent | Avoid leak in error path of asn1_parse2 (diff) | |
| download | openssl-76123661a1db136b9ef368dc296a628818e7a4cc.tar.xz openssl-76123661a1db136b9ef368dc296a628818e7a4cc.zip | |
Change returned -2 to 0 in EVP_Digest{Sign,Verify}Init()
The returned -2 was to mark when these operations are unsupported.
However, that breaks away from the previous API and expectations, and
there's not enough justification for that not being zero.
Reviewed-by: Nicola Tuveri <nic.tuv@gmail.com>
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/10815)
Diffstat (limited to 'crypto/evp/m_sigver.c')
| -rw-r--r-- | crypto/evp/m_sigver.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/evp/m_sigver.c b/crypto/evp/m_sigver.c index faee507342..dbfa01b3ed 100644 --- a/crypto/evp/m_sigver.c +++ b/crypto/evp/m_sigver.c @@ -167,7 +167,7 @@ static int do_sigver_init(EVP_MD_CTX *ctx, EVP_PKEY_CTX **pctx, legacy: if (ctx->pctx->pmeth == NULL) { EVPerr(0, EVP_R_OPERATION_NOT_SUPPORTED_FOR_THIS_KEYTYPE); - return -2; + return 0; } if (!(ctx->pctx->pmeth->flags & EVP_PKEY_FLAG_SIGCTX_CUSTOM)) { |