From: Amos Jeffries <squid3@treenet.co.nz>
Date: Tue, 30 Jul 2013 00:44:04 +0000 (-0600)
Subject: Fix debugs when refresh_pattern is missing the regex pattern
X-Git-Tag: SQUID_3_5_0_1~677
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2f3c75c33ea8be300bdce0a30a1ebd269eb6b804;p=thirdparty%2Fsquid.git

Fix debugs when refresh_pattern is missing the regex pattern

This is a very rare situation and may not be completely required. But it
did hinder debugging of refresh_pattern when the regex field token was
broken by parser updates.
---

diff --git a/src/cache_cf.cc b/src/cache_cf.cc
index 82a7fe2363..521c9dd5f5 100644
--- a/src/cache_cf.cc
+++ b/src/cache_cf.cc
@@ -2804,20 +2804,20 @@ parse_refreshpattern(RefreshPattern ** head)
     int errcode;
     int flags = REG_EXTENDED | REG_NOSUB;
 
-    if ((token = ConfigParser::NextToken()) == NULL) {
-        self_destruct();
-        return;
-    }
+    if ((token = ConfigParser::NextToken()) != NULL) {
+
+        if (strcmp(token, "-i") == 0) {
+            flags |= REG_ICASE;
+            token = ConfigParser::NextToken();
+        } else if (strcmp(token, "+i") == 0) {
+            flags &= ~REG_ICASE;
+            token = ConfigParser::NextToken();
+        }
 
-    if (strcmp(token, "-i") == 0) {
-        flags |= REG_ICASE;
-        token = ConfigParser::NextToken();
-    } else if (strcmp(token, "+i") == 0) {
-        flags &= ~REG_ICASE;
-        token = ConfigParser::NextToken();
     }
 
     if (token == NULL) {
+        debugs(3, DBG_CRITICAL, "FATAL: refresh_pattern missing the regex pattern parameter");
         self_destruct();
         return;
     }