From: Paul J. Reder <rederpj@apache.org>
Date: Fri, 21 Nov 2003 23:37:05 +0000 (+0000)
Subject: mod_expires: Don't report an Internal Server error if ExpiresDefault
X-Git-Tag: pre_ajp_proxy~1025
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=e030a58be62af56e860434fbb309d96acb0198a7;p=apache

mod_expires: Don't report an Internal Server error if ExpiresDefault
is used and has not been set in the httpd.conf file. PR: 23748, 24459

Submitted: Liam Quinn  <liam htmlhelp.com>
Reviewed: Paul J. Reder


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

diff --git a/CHANGES b/CHANGES
index b8703b8106..d3bc1a0e45 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,10 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) mod_expires: Don't report an Internal Server error if ExpiresDefault
+     is used and has not been set in the httpd.conf file. PR: 23748, 24459
+     [Liam Quinn  <liam htmlhelp.com>]
+
   *) mod_logio: Account for some bytes handed to the network layer prior to
      dropped connections.  [Jeff Trawick]
 
diff --git a/modules/metadata/mod_expires.c b/modules/metadata/mod_expires.c
index 47530a34c9..d7445ad634 100644
--- a/modules/metadata/mod_expires.c
+++ b/modules/metadata/mod_expires.c
@@ -548,7 +548,8 @@ static apr_status_t expires_filter(ap_filter_t *f,
                 expiry = conf->expiresdefault;
             }
         }
-        if (expiry != NULL) {
+        /* != '\0' covers the case where ExpiresDefault is not user defined. */
+        if (expiry != NULL && expiry[0] != '\0') {
             set_expiration_fields(r, expiry, t);
         }
     }