/[Apache-SVN]
ViewVC logotype

Revision 1718324


Jump to revision: Previous Next
Author: jim
Date: Mon Dec 7 12:17:35 2015 UTC (8 years, 6 months ago)
Changed paths: 6
Log Message:
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


Changed paths

Path Details
Directoryhttpd/httpd/branches/2.4.x/ modified , props changed
Directoryhttpd/httpd/branches/2.4.x/CHANGES modified , text changed
Directoryhttpd/httpd/branches/2.4.x/STATUS modified , text changed
Directoryhttpd/httpd/branches/2.4.x/docs/manual/mod/mod_proxy.xml modified , text changed
Directoryhttpd/httpd/branches/2.4.x/docs/manual/mod/mod_proxy_fdpass.xml modified , text changed
Directoryhttpd/httpd/branches/2.4.x/modules/proxy/mod_proxy_fdpass.c modified , text changed

infrastructure at apache.org
ViewVC Help
Powered by ViewVC 1.1.26