From: Colm MacCarthaigh <colm@apache.org>
Date: Tue, 18 Apr 2006 22:51:55 +0000 (+0000)
Subject: Correctly handle internal redirects, by testing only for r-main == NULL.
X-Git-Tag: 2.3.0~2445
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=acdbd4f887441f497d65c987144e654708b7b1c0;p=apache

Correctly handle internal redirects, by testing only for r-main == NULL.
ap_is_initial_req() returns false for internal redirects.


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

diff --git a/CHANGES b/CHANGES
index 68f4c855d1..0e6ee2f7fb 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@
 Changes with Apache 2.3.0
   [Remove entries to the current 2.0 and 2.2 section below, when backported]
 
+  *) mod_deflate: Allow mod_deflate to handle internal redirects.
+     [ Brian J. France <list firehawksystems.com>] 
+
   *) HTML-escape the Expect error message.  Not classed as security as
      an attacker has no way to influence the Expect header a victim will
      send to a target site.  Reported by Thiago Zaninotti
diff --git a/modules/filters/mod_deflate.c b/modules/filters/mod_deflate.c
index 4a9ae9831a..a447ebc5e6 100644
--- a/modules/filters/mod_deflate.c
+++ b/modules/filters/mod_deflate.c
@@ -240,7 +240,7 @@ static apr_status_t deflate_out_filter(ap_filter_t *f,
         const char *encoding;
 
         /* only work on main request/no subrequests */
-        if (!ap_is_initial_req(r)) {
+        if (r->main != NULL) {
             ap_remove_output_filter(f);
             return ap_pass_brigade(f->next, bb);
         }