httpd-cvs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From n..@apache.org
Subject svn commit: r1034918 - in /httpd/httpd/trunk: CHANGES modules/http/http_filters.c
Date Sun, 14 Nov 2010 00:17:29 GMT
Author: niq
Date: Sun Nov 14 00:17:29 2010
New Revision: 1034918

URL: http://svn.apache.org/viewvc?rev=1034918&view=rev
Log:
Return 400 not 503 if we have to abort due to malformed chunked encoding.

Modified:
    httpd/httpd/trunk/CHANGES
    httpd/httpd/trunk/modules/http/http_filters.c

Modified: httpd/httpd/trunk/CHANGES
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/CHANGES?rev=1034918&r1=1034917&r2=1034918&view=diff
==============================================================================
--- httpd/httpd/trunk/CHANGES [utf-8] (original)
+++ httpd/httpd/trunk/CHANGES [utf-8] Sun Nov 14 00:17:29 2010
@@ -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]

Modified: httpd/httpd/trunk/modules/http/http_filters.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/http/http_filters.c?rev=1034918&r1=1034917&r2=1034918&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/http/http_filters.c (original)
+++ httpd/httpd/trunk/modules/http/http_filters.c Sun Nov 14 00:17:29 2010
@@ -378,7 +378,7 @@ apr_status_t ap_http_filter(ap_filter_t 
                     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 
                             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;
                             }
                         }
                     }



Mime
View raw message