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);