svn commit: r155213 - in httpd/httpd/branches/2.0.x: STATUS server/mpm_common.c

[jo...@apache.org]

Author: jorton
Date: Thu Feb 24 09:51:49 2005
New Revision: 155213

URL: http://svn.apache.org/viewcvs?view=rev&rev=155213
Log:
* server/mpm_common.c (ap_mpm_pod_open, dummy_connection): Use
ap_listeners->bind_addr for the dummy connection rather than doing a
name lookup on the first listener's hostname and using one of the
addresses returned.

* server/mpm_common.c (dummy_connection): Log the address if the
connect() fails.

Reviewed by: jorton, trawick, stoddard

Modified:
    httpd/httpd/branches/2.0.x/STATUS
    httpd/httpd/branches/2.0.x/server/mpm_common.c

Modified: httpd/httpd/branches/2.0.x/STATUS
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/2.0.x/STATUS?view=diff&r1=155212&r2=155213
==============================================================================
--- httpd/httpd/branches/2.0.x/STATUS (original)
+++ httpd/httpd/branches/2.0.x/STATUS Thu Feb 24 09:51:49 2005
@@ -100,14 +100,6 @@
        Patch for 2.0: http://www.apache.org/~pquerna/c-i-f-leak.patch
        +1: pquerna, trawick
 
-    *) dummy_connection tweak: directly use the address of the first
-       listener rather than extracting through getnameinfo(), and log
-       the address for better diagnostics in error cases.
-       http://svn.apache.org/viewcvs.cgi?rev=102869&view=rev
-       http://svn.apache.org/viewcvs.cgi?rev=105264&view=rev
-       rediffed for 2.0 as: http://www.apache.org/~jorton/ap_dummy.diff
-       +1: jorton, trawick, stoddard
-
     *) mod_ssl: Set r->user from SSLUsername earlier so that it's
        actually useful.
        http://svn.apache.org/viewcvs.cgi?rev=153280&view=rev

Modified: httpd/httpd/branches/2.0.x/server/mpm_common.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/branches/2.0.x/server/mpm_common.c?view=diff&r1=155212&r2=155213
==============================================================================
--- httpd/httpd/branches/2.0.x/server/mpm_common.c (original)
+++ httpd/httpd/branches/2.0.x/server/mpm_common.c Thu Feb 24 09:51:49 2005
@@ -368,9 +368,6 @@
     apr_file_pipe_timeout_set((*pod)->pod_in, 0);
     (*pod)->p = p;
 
-    apr_sockaddr_info_get(&(*pod)->sa, ap_listeners->bind_addr->hostname,
-                          APR_UNSPEC, ap_listeners->bind_addr->port, 0, p);
-
     /* close these before exec. */
     apr_file_unset_inherit((*pod)->pod_in);
     apr_file_unset_inherit((*pod)->pod_out);
@@ -446,7 +443,7 @@
         return rv;
     }
 
-    rv = apr_socket_create(&sock, pod->sa->family, SOCK_STREAM, p);
+    rv = apr_socket_create(&sock, ap_listeners->bind_addr->family, SOCK_STREAM, p);
     if (rv != APR_SUCCESS) {
         ap_log_error(APLOG_MARK, APLOG_WARNING, rv, ap_server_conf,
                      "get socket to connect to listener");
@@ -467,7 +464,7 @@
         return rv;
     }
 
-    rv = apr_connect(sock, pod->sa);
+    rv = apr_connect(sock, ap_listeners->bind_addr);
     if (rv != APR_SUCCESS) {
         int log_level = APLOG_WARNING;
 
@@ -480,7 +477,7 @@
         }
 
         ap_log_error(APLOG_MARK, log_level, rv, ap_server_conf,
-                     "connect to listener");
+                     "connect to listener on %pI", ap_listeners->bind_addr);
     }
 
     apr_socket_close(sock);