diff options
| author | Yann Ylavic <ylavic@apache.org> | 2015-04-16 14:08:25 +0200 |
|---|---|---|
| committer | Yann Ylavic <ylavic@apache.org> | 2015-04-16 14:08:25 +0200 |
| commit | 70a0b58c6b8e8ec285c2d374636166816a87eef9 (patch) | |
| tree | e3cc84948779923a855de6324458a4d53f6ede50 /modules/proxy/mod_proxy_ajp.c | |
| parent | Remove entries for fixes/features already merged to 2.4.x branch (diff) | |
| download | apache2-70a0b58c6b8e8ec285c2d374636166816a87eef9.tar.xz apache2-70a0b58c6b8e8ec285c2d374636166816a87eef9.zip | |
mod_proxy_ajp: Rename output_failed to client_failed in ap_proxy_ajp_request().
Since any read/write error is caught by this flag, it really means "dialog with
client <ip:port> failed" (as per the associated log message), whereas write to
client (output) errors need not be handled differently than any error occuring
after some bytes have already been sent to the client, which is the purpose of
the data_sent flag.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1674056 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to '')
| -rw-r--r-- | modules/proxy/mod_proxy_ajp.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/modules/proxy/mod_proxy_ajp.c b/modules/proxy/mod_proxy_ajp.c index bce7fac068..fadc656197 100644 --- a/modules/proxy/mod_proxy_ajp.c +++ b/modules/proxy/mod_proxy_ajp.c @@ -177,7 +177,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, apr_byte_t conn_reuse = 0; const char *tenc; int havebody = 1; - int output_failed = 0; + int client_failed = 0; int backend_failed = 0; apr_off_t bb_len; int data_sent = 0; @@ -397,7 +397,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, else if (status == AP_FILTER_ERROR) { rv = AP_FILTER_ERROR; } - output_failed = 1; + client_failed = 1; break; } bufsiz = maxsize; @@ -408,7 +408,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, ap_log_rerror(APLOG_MARK, APLOG_DEBUG, status, r, APLOGNO(00881) "apr_brigade_flatten failed"); rv = HTTP_INTERNAL_SERVER_ERROR; - output_failed = 1; + client_failed = 1; break; } } @@ -522,7 +522,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, "error processing body.%s", r->connection->aborted ? " Client aborted connection." : ""); - output_failed = 1; + client_failed = 1; } data_sent = 1; apr_brigade_cleanup(output_brigade); @@ -549,7 +549,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, output_brigade) != APR_SUCCESS) { ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(00888) "error processing end"); - output_failed = 1; + client_failed = 1; } /* XXX: what about flush here? See mod_jk */ data_sent = 1; @@ -572,8 +572,8 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, if (!headers_sent) { r->status = HTTP_BAD_REQUEST; } + client_failed = 1; /* return DONE */ - output_failed = 1; data_sent = 1; break; } @@ -582,8 +582,8 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, * We either have finished successfully or we failed. * So bail out */ - if ((result == CMD_AJP13_END_RESPONSE) || backend_failed - || output_failed) + if ((result == CMD_AJP13_END_RESPONSE) + || backend_failed || client_failed) break; /* read the response */ @@ -605,10 +605,10 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, */ apr_brigade_cleanup(output_brigade); - if (backend_failed || output_failed) { + if (backend_failed || client_failed) { ap_log_rerror(APLOG_MARK, APLOG_DEBUG, 0, r, APLOGNO(00890) - "Processing of request failed backend: %i, " - "output: %i", backend_failed, output_failed); + "Processing of request failed backend: %i, client: %i", + backend_failed, client_failed); /* We had a failure: Close connection to backend */ conn->close = 1; if (data_sent) { @@ -685,7 +685,7 @@ static int ap_proxy_ajp_request(apr_pool_t *p, request_rec *r, } } } - else if (output_failed) { + else if (client_failed) { int level = (r->connection->aborted) ? APLOG_DEBUG : APLOG_ERR; ap_log_rerror(APLOG_MARK, level, status, r, APLOGNO(02822) "dialog with client %pI failed", |