Greg Ames and I spent some time today looking at a core dump on
daedalus where httpd got confused about the boundary between request
body and the header for the following request.
It appears that with the input filter rewrite we stopped using HTTP_IN
for internal redirects. HTTP_IN is now added in ap_read_request()
instead of a more general place. I think we need it also in
internal_internal_redirect() or it is needed in some more general
place *or* when added in ap_read_request() NULL should be passed for
the request_rec parameter so that it is tied to the conn_rec. That
should make it available for the redirect too (though I don't know if
the right thing will happen for subrequests in that case).
Maybe Greg played with a fix already; I was lazy/hungry and bailed
out.
Thoughts?
to reproduce:
two requests on the connection
first is POST/PUT with body to URI which is redirected internally
you'll see that there is no HTTP_IN for handing out body
bytes... ap_get_client_block() will call CORE_IN directly
--
Jeff Trawick | [EMAIL PROTECTED] | PGP public key at web site:
http://www.geocities.com/SiliconValley/Park/9289/
Born in Roswell... married an alien...
