You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by mt...@apache.org on 2005/04/09 12:39:53 UTC
svn commit: r160677 - httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
Author: mturk
Date: Sat Apr 9 03:39:50 2005
New Revision: 160677
URL: http://svn.apache.org/viewcvs?view=rev&rev=160677
Log:
Return correct status codes in case of bad request, that
is probably too large for AJP protocol.
Modified:
httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
Modified: httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c
URL: http://svn.apache.org/viewcvs/httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c?view=diff&r1=160676&r2=160677
==============================================================================
--- httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c (original)
+++ httpd/httpd/trunk/modules/proxy/mod_proxy_ajp.c Sat Apr 9 03:39:50 2005
@@ -130,7 +130,10 @@
"proxy: AJP: request failed to %pI (%s)",
conn->worker->cp->addr,
conn->worker->hostname);
- return HTTP_SERVICE_UNAVAILABLE;
+ if (status == AJP_EOVERFLOW)
+ return HTTP_BAD_REQUEST;
+ else
+ return HTTP_SERVICE_UNAVAILABLE;
}
/* allocate an AJP message to store the data of the buckets */
@@ -138,7 +141,7 @@
if (status != APR_SUCCESS) {
ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, r->server,
"proxy: ajp_alloc_data_msg failed");
- return status;
+ return HTTP_INTERNAL_SERVER_ERROR;
}
/* read the first bloc of data */
input_brigade = apr_brigade_create(p, r->connection->bucket_alloc);