From: Ruediger Pluem <rpluem@apache.org>
Date: Mon, 19 May 2008 19:30:15 +0000 (+0000)
Subject: * Only send 100-continue if the status code so far indicates success.
X-Git-Tag: 2.3.0~607
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=58d2be7591d3e424129ad986cb5e1d48964c73f6;p=apache

* Only send 100-continue if the status code so far indicates success.

PR: 43711


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

diff --git a/CHANGES b/CHANGES
index be3b6027da..e2021099ef 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,7 +2,10 @@
 Changes with Apache 2.3.0
 [ When backported to 2.2.x, remove entry from this file ]
 
-  *) ab: Make ab.c compile on VC6. PR 45024  [Ruediger Pluem]
+  *) http_filters: Don't return 100-continue on redirects. PR 43711
+     [Ruediger Pluem]
+
+  *) ab: Make ab.c compile on VC6. PR 45024 [Ruediger Pluem]
 
   *) configure: Don't reject libtool 2.x
      PR 44817 [Arfrever Frehtes Taifersar Arahesis <Arfrever.FTA gmail.com>]
diff --git a/modules/http/http_filters.c b/modules/http/http_filters.c
index 7f8b9224e1..d48ba7e3f6 100644
--- a/modules/http/http_filters.c
+++ b/modules/http/http_filters.c
@@ -323,7 +323,7 @@ apr_status_t ap_http_filter(ap_filter_t *f, apr_bucket_brigade *b,
             (ctx->state == BODY_LENGTH && ctx->remaining > 0)) &&
             f->r->expecting_100 && f->r->proto_num >= HTTP_VERSION(1,1) &&
             !(f->r->eos_sent || f->r->bytes_sent)) {
-            if (ap_is_HTTP_CLIENT_ERROR(f->r->status)) {
+            if (!ap_is_HTTP_SUCCESS(f->r->status)) {
                 ctx->state = BODY_NONE;
                 ctx->eos_sent = 1;
             } else {