]> granicus.if.org Git - apache/commitdiff
mpm_winnt: remove duplication of ap_process_connection
authorJacob Champion <jchampion@apache.org>
Mon, 29 Aug 2016 23:56:20 +0000 (23:56 +0000)
committerJacob Champion <jchampion@apache.org>
Mon, 29 Aug 2016 23:56:20 +0000 (23:56 +0000)
Further follow-up to the previous commit: now that we no longer patch a
network bucket into the brigade, we can revert to calling
ap_process_connection() directly instead of duplicating its logic.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1758309 13f79535-47bb-0310-9956-ffa450edef68

server/mpm/winnt/child.c

index 5e771fd70a8fd1411050e59193f7a1bac1f16488..63aa81f931b0c0a9730a7652564c1e1a6cefec98 100644 (file)
@@ -764,7 +764,6 @@ static DWORD __stdcall worker_main(void *thread_num_val)
     winnt_conn_ctx_t *context = NULL;
     int thread_num = (int)thread_num_val;
     ap_sb_handle_t *sbh;
-    int rc;
     conn_rec *c;
     apr_int32_t disconnected;
 
@@ -805,19 +804,7 @@ static DWORD __stdcall worker_main(void *thread_num_val)
         apr_os_thread_put(&thd, &osthd, context->ptrans);
         c->current_thread = thd;
 
-        /* follow ap_process_connection(c, context->sock) logic
-         * as it left us no chance to reinject our first data bucket.
-         */
-        ap_update_vhost_given_ip(c);
-
-        rc = ap_run_pre_connection(c, context->sock);
-        if (rc != OK && rc != DONE) {
-            c->aborted = 1;
-        }
-
-        if (!c->aborted) {
-            ap_run_process_connection(c);
-        }
+        ap_process_connection(c, context->sock);
 
         apr_socket_opt_get(context->sock, APR_SO_DISCONNECTED, &disconnected);