Amplify comment on server requests for client certificates
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 21 Sep 2014 16:59:44 +0000 (17:59 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Thu, 25 Sep 2014 17:39:25 +0000 (18:39 +0100)
src/src/tls-openssl.c

index 7e424f4f1a0f878fea4bc883bcd7e550696957ac..cb2e94f94159f7627e97034f1f05569f7b2640f6 100644 (file)
@@ -1377,7 +1377,16 @@ if (expcerts != NULL && *expcerts != '\0')
       return tls_error(US"SSL_CTX_load_verify_locations", host, NULL);
 
     /* Load the list of CAs for which we will accept certs, for sending
       return tls_error(US"SSL_CTX_load_verify_locations", host, NULL);
 
     /* Load the list of CAs for which we will accept certs, for sending
-    to the client.  XXX only for file source, not dir? */
+    to the client.  This is only for the one-file tls_verify_certificates
+    variant.
+    If a list isn't loaded into the server, but
+    some verify locations are set, the server end appears to make
+    a wildcard reqest for client certs.
+    Meanwhile, the client library as deafult behaviour *ignores* the list
+    we send over the wire - see man SSL_CTX_set_client_cert_cb.
+    Because of this, and that the dir variant is likely only used for
+    the public-CA bundle (not for a private CA), not worth fixing.
+    */
     if (file != NULL)
       {
       STACK_OF(X509_NAME) * names = SSL_load_client_CA_file(CS file);
     if (file != NULL)
       {
       STACK_OF(X509_NAME) * names = SSL_load_client_CA_file(CS file);