From: Jim Jagielski <jim@apache.org>
Date: Tue, 14 Aug 2012 21:11:29 +0000 (+0000)
Subject: Merge r1367819 from trunk:
X-Git-Tag: 2.4.3~36
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=13ccf47781cb85f6025b64852b5cb80491674fe3;p=apache

Merge r1367819 from trunk:

initialize the protocol field when manufacturing an apr socket
lest accidential autodata contents leak into the apr_socket_t

(this doesn't address any known problem symptom)

Submitted by: trawick
Reviewed/backported by: jim


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1373109 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/STATUS b/STATUS
index 5a790029d8..21a0044a77 100644
--- a/STATUS
+++ b/STATUS
@@ -94,20 +94,6 @@ PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
      +1: rjung, humbedooh, trawick (needs CHANGES entry)
      -0: What should CHANGES say???
 
-   * PR 52476: SSL failures on Windows with AcceptFilter https none
-     trunk patch: http://svn.apache.org/viewvc?rev=1371801&view=rev
-     2.4.x patch: trunk patch works
-     Questions/explanations: http://mail-archives.apache.org/mod_mbox/httpd-dev/201208.mbox/%3C20120813123221.GA21932%40redhat.com%3E
-     +1: trawick, gsmith, jim
-
-   * mpm_winnt: initialize the protocol field when manufacturing an apr socket
-     lest accidential autodata contents leak into the apr_socket_t.
-     (this doesn't address any known problem symptom)
-     trunk patch: http://svn.apache.org/viewvc?view=rev&rev=1367819
-     2.4.x patch: trunk patch works
-     +1: rjung, trawick, jim
-
-
 PATCHES PROPOSED TO BACKPORT FROM TRUNK:
   [ New proposals should be added at the end of the list ]
 
diff --git a/server/mpm/winnt/child.c b/server/mpm/winnt/child.c
index 8859fac319..ae34b07bb6 100644
--- a/server/mpm/winnt/child.c
+++ b/server/mpm/winnt/child.c
@@ -649,11 +649,12 @@ reinit: /* target of data or connect upon too many AcceptEx failures */
             }
         }
 
-        sockinfo.os_sock = &context->accept_socket;
-        sockinfo.local   = context->sa_server;
-        sockinfo.remote  = context->sa_client;
-        sockinfo.family  = context->sa_server->sa_family;
-        sockinfo.type    = SOCK_STREAM;
+        sockinfo.os_sock  = &context->accept_socket;
+        sockinfo.local    = context->sa_server;
+        sockinfo.remote   = context->sa_client;
+        sockinfo.family   = context->sa_server->sa_family;
+        sockinfo.type     = SOCK_STREAM;
+        sockinfo.protocol = IPPROTO_TCP;
         /* Restore the state corresponding to apr_os_sock_make's default
          * assumption of timeout -1 (really, a flaw of os_sock_make and
          * os_sock_put that it does not query to determine ->timeout).