Pull Andreas Metzler's fix for gnutls_certificate_verify_peers (bug 1095)
authorPhil Pennock <pdp@exim.org>
Sat, 24 Sep 2011 07:09:44 +0000 (03:09 -0400)
committerPhil Pennock <pdp@exim.org>
Sat, 24 Sep 2011 07:09:44 +0000 (03:09 -0400)
doc/doc-txt/ChangeLog
src/src/tls-gnu.c

index c1362b1bd5d131ecebe4a3ae81c1cab757d17e71..e58136040aa5a68ee49453dc91ae0e6a89b047f6 100644 (file)
@@ -114,6 +114,8 @@ PP/09 Handle IPv6 addresses with SPF.
 
 PP/10 GnuTLS: support TLS 1.2 & 1.1.
       Bugzilla 1156.
+      Use gnutls_certificate_verify_peers2() [patch from Andreas Metzler].
+      Bugzilla 1095.
 
 
 Exim version 4.76
index 4de9d4f688343bc96b6cd865e6462ef09387980f..6b80637e910973bff232222ac3c0400e97027f53 100644 (file)
@@ -235,10 +235,10 @@ Returns:     TRUE/FALSE
 static BOOL
 verify_certificate(gnutls_session session, const char **error)
 {
-int verify;
+int rc;
 uschar *dn_string = US"";
 const gnutls_datum *cert;
-unsigned int cert_size = 0;
+unsigned int verify, cert_size = 0;
 
 *error = NULL;
 
@@ -262,7 +262,7 @@ if (cert != NULL)
       dn_string = string_copy_malloc(buff);
     }
 
-  verify = gnutls_certificate_verify_peers(session);
+  rc = gnutls_certificate_verify_peers2(session, &verify);
   }
 else
   {
@@ -274,7 +274,7 @@ else
 /* Handle the result of verification. INVALID seems to be set as well
 as REVOKED, but leave the test for both. */
 
-if ((verify & (GNUTLS_CERT_INVALID|GNUTLS_CERT_REVOKED)) != 0)
+if ((rc < 0) || (verify & (GNUTLS_CERT_INVALID|GNUTLS_CERT_REVOKED)) != 0)
   {
   tls_certificate_verified = FALSE;
   if (*error == NULL) *error = ((verify & GNUTLS_CERT_REVOKED) != 0)?