svn commit: r1585054 - /httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c

[tr...@apache.org]

Author: trawick
Date: Sat Apr  5 11:45:47 2014
New Revision: 1585054

URL: http://svn.apache.org/r1585054
Log:
axe unnecessary preparation for some other module using
[ap_]connect_to_peer() and passing in its own module
identifier

Pointed out by: jailletc36

Modified:
    httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c

Modified: httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c?rev=1585054&r1=1585053&r2=1585054&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c (original)
+++ httpd/httpd/trunk/modules/aaa/mod_authnz_fcgi.c Sat Apr  5 11:45:47 2014
@@ -79,12 +79,10 @@ static apr_hash_t *fcgi_authn_providers,
  */
 static apr_status_t connect_to_peer(apr_socket_t **newsock,
                                     request_rec *r,
-                                    int module_index,
                                     apr_sockaddr_t *backend_addrs,
                                     const char *backend_name,
                                     apr_interval_time_t timeout)
 {
-#define FN_LOG_MARK __FILE__,__LINE__,module_index
     apr_status_t rv = APR_EINVAL; /* returned if no backend addr was provided
                                    */
     int connected = 0;
@@ -95,7 +93,7 @@ static apr_status_t connect_to_peer(apr_
         rv = apr_socket_create(newsock, addr->family,
                                SOCK_STREAM, 0, r->pool);
         if (rv != APR_SUCCESS) {
-            ap_log_rerror(FN_LOG_MARK, loglevel, rv, r,
+            ap_log_rerror(APLOG_MARK, loglevel, rv, r,
                           APLOGNO(02494) "error creating family %d socket "
                           "for target %s",
                           addr->family, backend_name);
@@ -110,7 +108,7 @@ static apr_status_t connect_to_peer(apr_
         rv = apr_socket_connect(*newsock, addr);
         if (rv != APR_SUCCESS) {
             apr_socket_close(*newsock);
-            ap_log_rerror(FN_LOG_MARK, loglevel, rv, r,
+            ap_log_rerror(APLOG_MARK, loglevel, rv, r,
                           APLOGNO(02495) "attempt to connect to %pI (%s) "
                           "failed", addr, backend_name);
             addr = addr->next;
@@ -715,7 +713,7 @@ static void req_rsp(request_rec *r, cons
 
     setupenv(r, password, apache_role);
 
-    rv = connect_to_peer(&s, r, APLOG_MODULE_INDEX, conf->backend_addrs, 
+    rv = connect_to_peer(&s, r, conf->backend_addrs,
                          conf->backend, FCGI_IO_TIMEOUT);
     if (rv == APR_SUCCESS) {
         apr_uint16_t request_id = 1;