diff options
author | Jacob Champion <jchampion@apache.org> | 2017-04-19 02:23:48 +0000 |
---|---|---|
committer | Jacob Champion <jchampion@apache.org> | 2017-04-19 02:23:48 +0000 |
commit | bac77f2026d9776bd462928443ea7dcedceb7458 (patch) | |
tree | a7fd5f67111f6d60cdceea3856ce63d07863f69f | |
parent | e8c348af4a3ba893cbf20231c6e5a29bae170aee (diff) | |
download | httpd-bac77f2026d9776bd462928443ea7dcedceb7458.tar.gz |
ssl_util.c: log which threadid callback is in use
This should help during testing and debugging. Use APLOG_NOTICE for now,
until we fix PR60999, so that people can actually see the log message.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/trunk-openssl-threadid@1791847 13f79535-47bb-0310-9956-ffa450edef68
-rw-r--r-- | modules/ssl/ssl_util.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/modules/ssl/ssl_util.c b/modules/ssl/ssl_util.c index 1ae93acd19..0e0a2694e4 100644 --- a/modules/ssl/ssl_util.c +++ b/modules/ssl/ssl_util.c @@ -373,7 +373,11 @@ static void ssl_dyn_destroy_function(struct CRYPTO_dynlock_value *l, #if DEFAULT_THREADID_IS_SAFE /* We don't need to set up a threadid callback on this platform. */ -void ssl_util_thread_id_setup(apr_pool_t *p) { } +void ssl_util_thread_id_setup(apr_pool_t *p) +{ + ap_log_perror(APLOG_MARK, APLOG_NOTICE, 0, p, APLOGNO() + "using builtin threadid callback for OpenSSL"); +} #else @@ -434,10 +438,16 @@ static apr_status_t ssl_util_thr_id_cleanup(void *old) void ssl_util_thread_id_setup(apr_pool_t *p) { #if HAVE_CRYPTO_SET_ID_CALLBACK + ap_log_perror(APLOG_MARK, APLOG_NOTICE, 0, p, APLOGNO() + "using deprecated CRYPTO_set_id_callback for OpenSSL"); + /* This API is deprecated, but we prefer it to its replacement since it * allows us to unset the callback when this module is being unloaded. */ CRYPTO_set_id_callback(ssl_util_thr_id); #else + ap_log_perror(APLOG_MARK, APLOG_NOTICE, 0, p, APLOGNO() + "using dangerous CRYPTO_THREADID_set_callback for OpenSSL"); + /* This is a last resort. We can only set this once, which means that we'd * better not get loaded into a different address during a restart. See * PR60947. */ |