You are viewing a plain text version of this content. The canonical link for it is here.

Posted to cvs@httpd.apache.org by yl...@apache.org on 2015/03/25 16:28:27 UTC

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

Author: ylavic
Date: Wed Mar 25 15:28:27 2015
New Revision: 1669133

URL: http://svn.apache.org/r1669133
Log:
Update mod_proxy-transfer patch to address Eric's concerns.

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=1669133&r1=1669132&r2=1669133&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Wed Mar 25 15:28:27 2015
@@ -311,11 +311,13 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK:
      side when none occurs next on the other side, and abort.
      trunk patch: http://svn.apache.org/r1657636
                   http://svn.apache.org/r1657638
-     2.4.x patch: http://people.apache.org/~ylavic/httpd-2.4.x-mod_proxy-transfer.patch
+                  http://svn.apache.org/r1669130
+     2.4.x patch: http://people.apache.org/~ylavic/httpd-2.4.x-mod_proxy-transfer-v2.patch
      +1: ylavic
      covener: I noticed while reviewing that rv is not used in ap_log_rerror at least
      in the path context. If it had, the |= would have dropped the strerror.  Think
      there's any risk in having rv like that floating around?
+     ylavic: Agreed, fixed in r1669130 (and v2) by using a pure boolean instead.
 
   *) mod_dav: Avoid a potential integer underflow in the lock timeout value sent
      back to a client. PR 55420