www/apache
1
0
Fork 0
mirror of https://github.com/apache/httpd.git synced 2025-08-07 04:02:58 +03:00
Code Activity

fix weird things that happen with canned error messages due to using two

Browse Source 
different request_recs after an ErrorDocument internal redirect failure.
examples: wrong Content-Type, garbled output from ebcdic servers due to
double charset translation


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@96364 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Greg Ames
2002-08-13 14:27:39 +00:00
parent cdf30bf3e6
commit 65b5ceef11
3 changed files with 40 additions and 51 deletions
Download Patch File Download Diff File Expand all files Collapse all files

67
modules/http/http_request.c
View File

@@ -96,6 +96,23 @@
* Mainline request processing...
*/
/* XXX A cleaner and faster way to do this might be to pass the request_rec
* down the filter chain as a parameter. It would need to change for
* subrequest vs. main request filters; perhaps the subrequest filter could
* make the switch.
*/
static void update_r_in_filters(ap_filter_t *f,
request_rec *from,
request_rec *to)
{
while (f) {
if (f->r == from) {
f->r = to;
}
f = f->next;
}
}
AP_DECLARE(void) ap_die(int type, request_rec *r)
{
int error_index = ap_index_of_response(type);
@@ -124,6 +141,20 @@ AP_DECLARE(void) ap_die(int type, request_rec *r)
while (r_1st_err->prev && (r_1st_err->prev->status != HTTP_OK))
r_1st_err = r_1st_err->prev; /* Get back to original error */
if (r_1st_err != r) {
/* The recursive error was caused by an ErrorDocument specifying
* an internal redirect to a bad URI. ap_internal_redirect has
* changed the filter chains to point to the ErrorDocument's
* request_rec. Back out those changes so we can safely use the
* original failing request_rec to send the canned error message.
*
* ap_send_error_response gets rid of existing resource filters
* on the output side, so we can skip those.
*/
update_r_in_filters(r_1st_err->proto_output_filters, r, r_1st_err);
update_r_in_filters(r_1st_err->input_filters, r, r_1st_err);
}
custom_response = NULL; /* Do NOT retry the custom thing! */
}
@@ -301,7 +332,6 @@ static apr_table_t *rename_original_env(apr_pool_t *p, apr_table_t *t)
static request_rec *internal_internal_redirect(const char *new_uri,
request_rec *r) {
int access_status;
ap_filter_t *f;
request_rec *new = (request_rec *) apr_pcalloc(r->pool,
sizeof(request_rec));
@@ -367,21 +397,8 @@ static request_rec *internal_internal_redirect(const char *new_uri,
new->output_filters = new->proto_output_filters;
new->input_filters = new->proto_input_filters;
f = new->input_filters;
while (f) {
if (f->r == r) {
f->r = new;
}
f = f->next;
}
f = new->output_filters;
while (f) {
if (f->r == r) {
f->r = new;
}
f = f->next;
}
update_r_in_filters(new->input_filters, r, new);
update_r_in_filters(new->output_filters, r, new);
apr_table_setn(new->subprocess_env, "REDIRECT_STATUS",
apr_itoa(r->pool, r->status));
@@ -402,8 +419,6 @@ static request_rec *internal_internal_redirect(const char *new_uri,
/* XXX: Is this function is so bogus and fragile that we deep-6 it? */
AP_DECLARE(void) ap_internal_fast_redirect(request_rec *rr, request_rec *r)
{
ap_filter_t *filters;
/* We need to tell POOL_DEBUG that we're guaranteeing that rr->pool
* will exist as long as r->pool. Otherwise we run into troubles because
* some values in this request will be allocated in r->pool, and others in
@@ -449,20 +464,8 @@ AP_DECLARE(void) ap_internal_fast_redirect(request_rec *rr, request_rec *r)
* their f->r structure when it is pointing to rr, the real request_rec
* will not get updated. Fix that here.
*/
filters = r->input_filters;
while (filters) {
if (filters->r == rr) {
filters->r = r;
}
filters = filters->next;
}
filters = r->output_filters;
while (filters) {
if (filters->r == rr) {
filters->r = r;
}
filters = filters->next;
}
update_r_in_filters(r->input_filters, rr, r);
update_r_in_filters(r->output_filters, rr, r);
}
AP_DECLARE(void) ap_internal_redirect(const char *new_uri, request_rec *r)