Bug #1166: Configuration confusing on empty ACLS

Still some bugfixing is needed

diff --git a/src/ACLARP.cc b/src/ACLARP.cc
index bee51796d725441a1f034545f7bb1d86e08889dd..9fb7187299fca8233846f8fa0ad40848a04726fa 100644 (file)
--- a/src/ACLARP.cc
+++ b/src/ACLARP.cc
@@ -1,5 +1,5 @@
 /*
- * $Id: ACLARP.cc,v 1.15 2005/05/08 06:53:58 hno Exp $
+ * $Id: ACLARP.cc,v 1.16 2005/05/08 09:15:38 serassio Exp $
  *
  * DEBUG: section 28    Access Control
  * AUTHOR: Duane Wessels
@@ -120,7 +120,7 @@ ACLARP::typeString() const
 bool
 ACLARP::empty () const
 {
-    return data->empty();
+    return data == NULL;
 }
 
 /* ==== BEGIN ARP ACL SUPPORT ============================================= */

diff --git a/src/ACLASN.h b/src/ACLASN.h
index 7733925d1473c1bbea0d4c89005f60cfd8e63832..02da65a86a9a6a791ec3d234e834107f7ece9bfd 100644 (file)
--- a/src/ACLASN.h
+++ b/src/ACLASN.h
@@ -1,6 +1,6 @@
 
 /*
- * $Id: ACLASN.h,v 1.6 2005/04/18 21:52:41 hno Exp $
+ * $Id: ACLASN.h,v 1.7 2005/05/08 09:15:39 serassio Exp $
  *
  *
  * SQUID Web Proxy Cache          http://www.squid-cache.org/
@@ -55,6 +55,7 @@ public:
     virtual bool match(struct IN_ADDR);
     virtual wordlist *dump();
     virtual void parse();
+    bool empty() const;
     virtual ACLData<struct IN_ADDR> *clone() const;
     virtual void prepareForUse();
 

diff --git a/src/ACLIP.cc b/src/ACLIP.cc
index 0d3a5c856fdf7eb6a8fbd148989279f46252aa88..3ca6fc758530fb5754836c16dd5c57c28237df2f 100644 (file)
--- a/src/ACLIP.cc
+++ b/src/ACLIP.cc
@@ -355,7 +355,7 @@ ACLIP::dump() const
 bool
 ACLIP::empty () const
 {
-    return data->empty();
+    return data == NULL;
 }
 
 int

diff --git a/src/ACLIntRange.cc b/src/ACLIntRange.cc
index 0d3f4994498e46ecef0808b5d98d9ed2da1e14ea..4ff97466ba4f165005c4f0766c55cdf156c8b633 100644 (file)
--- a/src/ACLIntRange.cc
+++ b/src/ACLIntRange.cc
@@ -1,5 +1,5 @@
 /*
- * $Id: ACLIntRange.cc,v 1.4 2005/05/08 06:36:45 hno Exp $
+ * $Id: ACLIntRange.cc,v 1.5 2005/05/08 09:15:39 serassio Exp $
  *
  * DEBUG: section 28    Access Control
  * AUTHOR: Robert Collins
@@ -73,7 +73,7 @@ ACLIntRange::parse()
 bool
 ACLIntRange::empty() const
 {
-    return ranges != NULL;
+    return ranges == NULL;
 }
 
 bool

diff --git a/src/ACLMethodData.cc b/src/ACLMethodData.cc
index 193386c66f58242bfa9acffc7494b1a059e1ecb0..a86b1d61cc5eec8ce4c19aa4d50bf535bd987601 100644 (file)
--- a/src/ACLMethodData.cc
+++ b/src/ACLMethodData.cc
@@ -1,5 +1,5 @@
 /*
- * $Id: ACLMethodData.cc,v 1.6 2005/05/08 06:36:45 hno Exp $
+ * $Id: ACLMethodData.cc,v 1.7 2005/05/08 09:15:39 serassio Exp $
  *
  * DEBUG: section 28    Access Control
  * AUTHOR: Duane Wessels
@@ -96,7 +96,7 @@ ACLMethodData::parse()
 bool
 ACLMethodData::empty() const
 {
-    return values != NULL;
+    return values == NULL;
 }
 
 ACLData<method_t> *

diff --git a/src/ACLUserData.cc b/src/ACLUserData.cc
index 66c5acc3939743b360cc33de285919ea1021df5d..69112e1c99ff1f3674df7daa56c228640b292efa 100644 (file)
--- a/src/ACLUserData.cc
+++ b/src/ACLUserData.cc
@@ -166,7 +166,7 @@ ACLUserData::parse()
 bool
 ACLUserData::empty() const
 {
-    return names == NULL;
+    return ((names == NULL) && (flags.required != 1));
 }
 
 ACLData<char const *> *

diff --git a/src/asn.cc b/src/asn.cc
index 62eee5f534c3519d7a9753eff3ea908619f95a1c..b17003d6d0b4cea59787024c5c9b72d4f1d4d641 100644 (file)
--- a/src/asn.cc
+++ b/src/asn.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: asn.cc,v 1.100 2005/04/18 21:52:42 hno Exp $
+ * $Id: asn.cc,v 1.101 2005/05/08 09:15:39 serassio Exp $
  *
  * DEBUG: section 53    AS Number handling
  * AUTHOR: Duane Wessels, Kostas Anagnostakis
@@ -588,6 +588,12 @@ ACLASN::dump()
     return W;
 }
 
+bool
+ACLASN::empty () const
+{
+    return data == NULL;
+}
+
 void
 ACLASN::parse()
 {