Return 400 not 503 if we have to abort due to malformed chunked encoding.

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1034918 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/CHANGES b/CHANGES
index 6963dba9d69a06104c662426db4d6ef61e9e9fcb..f83db1f2e1a549ba03775d5b8eb80086a5a3559f 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -248,6 +248,9 @@ Changes with Apache 2.3.9
       Aron Ujvari <xanco nikhok.hu>, Aleksey Midenkov <asm uezku.kemsu.ru>,
       <dan listening-station.net; trunk version Nick Kew]
 
+  *) HTTP protocol: return 400 not 503 if we have to abort due to malformed
+     chunked encoding. [Nick Kew]
+
 Changes with Apache 2.3.8
 
   *) suexec: Support large log files. PR 45856. [Stefan Fritsch]

diff --git a/modules/http/http_filters.c b/modules/http/http_filters.c
index 06ef6b3ce5580592d945eef2bbc741ab1b0a5d18..45d9018f50ab895747d846a8e45c979808ae71cb 100644 (file)
--- a/modules/http/http_filters.c
+++ b/modules/http/http_filters.c
@@ -378,7 +378,7 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
                     ctx->remaining = get_chunk_size(ctx->chunk_ln);
                     if (ctx->remaining == INVALID_CHAR) {
                         rv = APR_EGENERAL;
-                        http_error = HTTP_SERVICE_UNAVAILABLE;
+                        http_error = HTTP_BAD_REQUEST;
                     }
                 }
             }
@@ -483,7 +483,7 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
                             ctx->remaining = get_chunk_size(ctx->chunk_ln);
                             if (ctx->remaining == INVALID_CHAR) {
                                 rv = APR_EGENERAL;
-                                http_error = HTTP_SERVICE_UNAVAILABLE;
+                                http_error = HTTP_BAD_REQUEST;
                             }
                         }
                     }