mirror of
https://github.com/apache/httpd.git
synced 2025-08-08 15:02:10 +03:00
Remove code to support per-dir-CA configuration which relies on a
function which was never included in OpenSSL: * acinclude.m4: Remove test for fictional function. * modules/ssl/ssl_engine_kernel.c (ssl_hook_Access): Remove code. * modules/ssl/ssl_engine_config.c (MODSSL_SET_CA): Remove unused macro. (ssl_cmd_SSLCACertificatePath, ssl_cmd_SSLCACertificateFile): Fail if per-dir-CA is configured. git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@757380 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
@@ -540,67 +540,6 @@ int ssl_hook_Access(request_rec *r)
|
||||
}
|
||||
}
|
||||
|
||||
/*
|
||||
* override SSLCACertificateFile & SSLCACertificatePath
|
||||
* This is only enabled if the SSL_set_cert_store() function
|
||||
* is available in the ssl library. the 1.x based mod_ssl
|
||||
* used SSL_CTX_set_cert_store which is not thread safe.
|
||||
*/
|
||||
|
||||
#ifdef HAVE_SSL_SET_CERT_STORE
|
||||
/*
|
||||
* check if per-dir and per-server config field are not the same.
|
||||
* if f is defined in per-dir and not defined in per-server
|
||||
* or f is defined in both but not the equal ...
|
||||
*/
|
||||
#define MODSSL_CFG_NE(f) \
|
||||
(dc->f && (!sc->f || (sc->f && strNE(dc->f, sc->f))))
|
||||
|
||||
#define MODSSL_CFG_CA(f) \
|
||||
(dc->f ? dc->f : sc->f)
|
||||
|
||||
if (MODSSL_CFG_NE(szCACertificateFile) ||
|
||||
MODSSL_CFG_NE(szCACertificatePath))
|
||||
{
|
||||
STACK_OF(X509_NAME) *ca_list;
|
||||
const char *ca_file = MODSSL_CFG_CA(szCACertificateFile);
|
||||
const char *ca_path = MODSSL_CFG_CA(szCACertificatePath);
|
||||
|
||||
cert_store = X509_STORE_new();
|
||||
|
||||
if (!X509_STORE_load_locations(cert_store, ca_file, ca_path)) {
|
||||
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,
|
||||
"Unable to reconfigure verify locations "
|
||||
"for client authentication");
|
||||
ssl_log_ssl_error(APLOG_MARK, APLOG_ERR, r->server);
|
||||
|
||||
X509_STORE_free(cert_store);
|
||||
|
||||
return HTTP_FORBIDDEN;
|
||||
}
|
||||
|
||||
/* SSL_free will free cert_store */
|
||||
SSL_set_cert_store(ssl, cert_store);
|
||||
|
||||
if (!(ca_list = ssl_init_FindCAList(r->server, r->pool,
|
||||
ca_file, ca_path)))
|
||||
{
|
||||
ap_log_error(APLOG_MARK, APLOG_ERR, 0, r->server,
|
||||
"Unable to determine list of available "
|
||||
"CA certificates for client authentication");
|
||||
|
||||
return HTTP_FORBIDDEN;
|
||||
}
|
||||
|
||||
SSL_set_client_CA_list(ssl, ca_list);
|
||||
renegotiate = TRUE;
|
||||
|
||||
ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r,
|
||||
"Changed client verification locations will force "
|
||||
"renegotiation");
|
||||
}
|
||||
#endif /* HAVE_SSL_SET_CERT_STORE */
|
||||
|
||||
/* If a renegotiation is now required for this location, and the
|
||||
* request includes a message body (and the client has not
|
||||
* requested a "100 Continue" response), then the client will be
|
||||
|
Reference in New Issue
Block a user