Commit 2d764632 authored by Yann Ylavic's avatar Yann Ylavic
Browse files

Update mod_proxy-transfer patch to v3 (v2 was missing APR_EBADF case). 

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

STATUS

+2 −2
Changes for STATUS: 2 added lines, 2 removed lines.
Original line number Diff line number Diff line
@@ -312,12 +312,12 @@ PATCHES PROPOSED TO BACKPORT FROM TRUNK: 
     trunk patch: http://svn.apache.org/r1657636

                  http://svn.apache.org/r1657638

                  http://svn.apache.org/r1669130

     2.4.x patch: http://people.apache.org/~ylavic/httpd-2.4.x-mod_proxy-transfer-v2.patch

     2.4.x patch: http://people.apache.org/~ylavic/httpd-2.4.x-mod_proxy-transfer-v3.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.

     ylavic: Agreed, fixed in r1669130 (and v3) 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