[PATCH] expressionlist ignored

[Keith Wannamaker]

This patch is against 1.1.4.

It turns out if the url or domain database
was empty, no further destination targest
would be consulted.  For instance,

dest {
  domainlist     foo/domain.db
  urllist        foo/url.db
  expressionlist foo/exp
}

If domain.db is empty, url.db and exp
will not be consulted.  Further, if
domain.db is nonempty but url.db is
empty, exp will not be consulted.

All in all super work, guys.

Keith
--- sg.y        Mon Mar 27 09:08:39 2000
+++ sg.y.new    Fri Sep 14 15:12:12 2001
@@ -2047,9 +2047,7 @@
       }
       if(aclpass->dest->domainlistDb != NULL){
        result = defined(aclpass->dest->domainlistDb, req->domain, &dbdata);
-       if(result == DB_NOTFOUND)
-         continue;
-       if(result){
+       if(result != DB_NOTFOUND && result){
          if(aclpass->access){
            access++;
            break; 
@@ -2061,9 +2059,7 @@
       }
       if(aclpass->dest->urllistDb != NULL && access){
        result = defined(aclpass->dest->urllistDb,req->strippedurl, &dbdata);
-       if(result == DB_NOTFOUND)
-         continue;
-       if(result){
+       if(result != DB_NOTFOUND && result){
          if(aclpass->access){
            access++;
            break;