Changeset 1359:3ea918ddb486

Timestamp:

08/03/13 23:24:26 (11 years ago)

Author:

Pascal Chevrel <pascal.chevrel@…>

Branch:

2.5

Message:

Ticket 1462: Don't check all DNSBL services for spam in isSpam() if we have a spam match on one server, just return true

File:

• plugins/antispam/filters/class.dc.filter.iplookup.php (modified) (1 diff)

plugins/antispam/filters/class.dc.filter.iplookup.php

@@ -41 +41 @@
      {
           if (!$ip || long2ip(ip2long($ip)) != $ip) {
-               return;
+               return false;
           }
-
-          $match = array();
 
           $bls = $this->getServers();
           $bls = preg_split('/\s*,\s*/',$bls);
 
-          foreach ($bls as $bl)
-          {
+          foreach ($bls as $bl) {
                if ($this->dnsblLookup($ip,$bl)) {
-                    $match[] = $bl;
+                    return true;
                }
           }
 
-          if (!empty($match)) {
-               $status = substr(implode(', ',$match),0,128);
-               return true;
-          }
+          return false;
      }