You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by jo...@apache.org on 2012/12/11 16:38:57 UTC

svn commit: r1420229 - /httpd/httpd/branches/2.4.x/STATUS

Author: jorton
Date: Tue Dec 11 15:38:54 2012
New Revision: 1420229

URL: http://svn.apache.org/viewvc?rev=1420229&view=rev
Log:
Update mod_proxy_ftp EPSV handling patch per sf's comment.

Modified:
    httpd/httpd/branches/2.4.x/STATUS

Modified: httpd/httpd/branches/2.4.x/STATUS
URL: http://svn.apache.org/viewvc/httpd/httpd/branches/2.4.x/STATUS?rev=1420229&r1=1420228&r2=1420229&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Tue Dec 11 15:38:54 2012
@@ -176,17 +176,10 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
    * mod_proxy_ftp: Fix segfaults on IPv4 requests to hosts with DNS AAAA records.
      PR 40841.
      trunk patch: http://svn.apache.org/viewvc?rev=1404625&view=rev
-     2.4.x patch: trunk patch works:
-     +1: jim, druggeri
-     sf notes: The if (!epsv_addr) check is wrong: the return value of
-               apr_sockaddr_info_get() should be checked instead, or
-               epsv_addr needs to be initialized to NULL.
-               The if (!data_sock) case can never be hit because
-               apr_socket_create() never returns a NULL socket when it also
-               returns APR_SUCCESS.  And if data_sock were NULL, the
-               apr_socket_opt_set() call would segfault before the check is
-               hit.
-     -1: sf (until at least the epsv_addr check is fixed)
+                  http://svn.apache.org/viewvc?rev=1420223&view=rev
+     2.4.x patch: trunk patch works, or
+                  http://people.apache.org/~jorton/proxyftpepsv-24.patch
+     +1: jorton, 
 
 A list of further possible backports can be found at:
     http://people.apache.org/~rjung/patches/possible-backports-httpd-trunk-2_4.txt