Le 18/01/2016 17:54, j...@apache.org a écrit :
Author: jim
Date: Mon Jan 18 16:54:12 2016
New Revision: 1725307
URL: http://svn.apache.org/viewvc?rev=1725307&view=rev
Log:
simplify and start on concept framework of OPTIONS
Modified:
httpd/httpd/trunk/modules/proxy/mod_proxy_hcheck.c
Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_hcheck.c
URL:
http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/proxy/mod_proxy_hcheck.c?rev=1725307&r1=1725306&r2=1725307&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_hcheck.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_hcheck.c Mon Jan 18 16:54:12 2016
[...]
@@ -324,13 +325,24 @@ static const char *set_hc_template(cmd_p
return NULL;
}
-static void backend_cleanup(const char *proxy_function, proxy_conn_rec
*backend,
- server_rec *s)
+static apr_status_t backend_cleanup(const char *proxy_function, proxy_conn_rec
*backend,
+ server_rec *s, int status)
{
if (backend) {
backend->close = 1;
ap_proxy_release_connection(proxy_function, backend, s);
}
+ ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s, APLOGNO()
+ "Health check %s Status (%d) for %s.",
+ ap_proxy_show_hcmethod(backend->worker->s->method),
This add a smatch warning because previous code (i.e. "if (backend)")
assumes that 'backend' can be NULL here.
+ status,
+ backend->worker->s->name);
+ if (status != OK) {
+ return APR_EGENERAL;
+ }
+ return APR_SUCCESS;
+
[...]
