diff options
author | Dr. Stephen Henson <steve@openssl.org> | 2009-01-07 10:48:23 +0000 |
---|---|---|
committer | Dr. Stephen Henson <steve@openssl.org> | 2009-01-07 10:48:23 +0000 |
commit | a00c3c4019a4bc0fa938b7dd3c0cd8e95cdc6943 (patch) | |
tree | 97dc675154f89679d944819529935c988924a301 /ssl/s2_clnt.c | |
parent | f4677b79609a17b125c46a8bc9899be94ac65919 (diff) |
Properly check EVP_VerifyFinal() and similar return values
(CVE-2008-5077).
Submitted by: Ben Laurie, Bodo Moeller, Google Security Team
Diffstat (limited to 'ssl/s2_clnt.c')
-rw-r--r-- | ssl/s2_clnt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ssl/s2_clnt.c b/ssl/s2_clnt.c index e2a90a3ca2..782129cd5d 100644 --- a/ssl/s2_clnt.c +++ b/ssl/s2_clnt.c @@ -1044,7 +1044,7 @@ int ssl2_set_certificate(SSL *s, int type, int len, const unsigned char *data) i=ssl_verify_cert_chain(s,sk); - if ((s->verify_mode != SSL_VERIFY_NONE) && (!i)) + if ((s->verify_mode != SSL_VERIFY_NONE) && (i <= 0)) { SSLerr(SSL_F_SSL2_SET_CERTIFICATE,SSL_R_CERTIFICATE_VERIFY_FAILED); goto err; |