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 17:04:11 UTC
svn commit: r1669141 - /httpd/httpd/branches/2.4.x/STATUS
Author: ylavic
Date: Wed Mar 25 16:04:11 2015
New Revision: 1669141
URL: http://svn.apache.org/r1669141
Log:
Update mod_proxy-transfer patch to v3 (v2 was missing APR_EBADF case).
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=1669141&r1=1669140&r2=1669141&view=diff
==============================================================================
--- httpd/httpd/branches/2.4.x/STATUS (original)
+++ httpd/httpd/branches/2.4.x/STATUS Wed Mar 25 16:04:11 2015
@@ -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