> if (ap_status_drops_connection(r->status) || > (r->main && ap_status_drops_connection(r->main->status))) { > return OK; > } > > The idea is that if our status code is such that we're trying to > avoid reading the body, we shouldn't actually read it. We need > the r->main trick as well because of subreqs (the .html.var file > is a subreq handled by default_handler, so it will call discard_body > as well on each subreq!).
Hmm, I may not be remembering this correctly, but is there any situation in which a subrequest would be allowed to call discard_body? If not, it can simply check if (r->main || ap_status_drops_connection(r->status)) { return OK; } ....Roy