You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by mi...@apache.org on 2010/10/18 21:04:21 UTC

svn commit: r1023952 - /httpd/httpd/trunk/modules/cache/mod_cache.c

Author: minfrin
Date: Mon Oct 18 19:04:21 2010
New Revision: 1023952

URL: http://svn.apache.org/viewvc?rev=1023952&view=rev
Log:
Remove the remove_url filter consistently for both the ap_die() and the
mod_proxy cases.

Modified:
    httpd/httpd/trunk/modules/cache/mod_cache.c

Modified: httpd/httpd/trunk/modules/cache/mod_cache.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/cache/mod_cache.c?rev=1023952&r1=1023951&r2=1023952&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/cache/mod_cache.c (original)
+++ httpd/httpd/trunk/modules/cache/mod_cache.c Mon Oct 18 19:04:21 2010
@@ -1595,6 +1595,8 @@ static void cache_insert_error_filter(re
     if (dummy) {
         cache_request_rec *cache = (cache_request_rec *) dummy;
 
+        ap_remove_output_filter(cache->remove_url_filter);
+
         if (cache->stale_handle && cache->save_filter
                 && !cache->stale_handle->cache_obj->info.control.must_revalidate
                 && !cache->stale_handle->cache_obj->info.control.proxy_revalidate) {
@@ -1627,8 +1629,6 @@ static void cache_insert_error_filter(re
                         "110 Response is stale");
             }
 
-            ap_remove_output_filter(cache->remove_url_filter);
-
             cache_run_cache_status(
                     cache->handle,
                     r,