]> git.ipfire.org Git - thirdparty/apache/httpd.git/commit
Merge r1717063 from trunk:
authorJim Jagielski <jim@apache.org>
Mon, 7 Dec 2015 12:17:35 +0000 (12:17 +0000)
committerJim Jagielski <jim@apache.org>
Mon, 7 Dec 2015 12:17:35 +0000 (12:17 +0000)
commitadea9cbfa554661b27ad6f3ad6bbc3184bd03099
treecb72fb26892b0d43f1e8d2cfaacbb4dddcdf2605
parent458cd6f35b0f507c5a1aab72c22431138c723f52
Merge r1717063 from trunk:

Fix the use of the default 'flush' provider.
Improve documentation for the "flusher" parameter.
Remove useless empty lines.

See http://mail-archives.apache.org/mod_mbox/httpd-dev/200812.mbox/%3C494226C0.4050407@force-elite.com%3E for some more explanation.
A python script is given there to test.
I had to tweak it to have it work
(use:
        fd, payload = passfd.recvfd(conn.fileno())
instead of:
        fd = passfd.recvfd(conn.fileno())
)

This is a r1058621 regression, where somehow "char *flusher" has been turned into a "char flusher[]". So it is been broken since the beginning of 2.4.x

After this change (i.e. r1058621), 'flusher' is no more a pointer (NULL'ed when the structure it belongs to is created) but the address of an array within a structure. It can not be NULL anymore.
So, we now have to look at the content of the array itself to see if it has been initialized or if we have to use the default value instead.
Submitted by: jailletc36
Reviewed/backported by: jim

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1718324 13f79535-47bb-0310-9956-ffa450edef68
CHANGES
STATUS
docs/manual/mod/mod_proxy.xml
docs/manual/mod/mod_proxy_fdpass.xml
modules/proxy/mod_proxy_fdpass.c