From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Wed, 24 Jul 2019 09:14:07 +0000 (+0100)
Subject: [Minor] Reputation: Check results before setting
X-Git-Tag: 2.0~502
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f95b1d975c9f768b386299a352a740d7a855a982;p=thirdparty%2Frspamd.git

[Minor] Reputation: Check results before setting
---

diff --git a/src/plugins/lua/reputation.lua b/src/plugins/lua/reputation.lua
index aa6697eab4..9e28e876bc 100644
--- a/src/plugins/lua/reputation.lua
+++ b/src/plugins/lua/reputation.lua
@@ -484,14 +484,16 @@ local function ip_reputation_idempotent(task, rule)
     end
   end
   local sc = extract_task_score(task, rule)
-  if asn then
-    rule.backend.set_token(task, rule, cfg.asn_prefix .. asn, sc)
-  end
-  if country then
-    rule.backend.set_token(task, rule, cfg.country_prefix .. country, sc)
-  end
+  if sc then
+    if asn then
+      rule.backend.set_token(task, rule, cfg.asn_prefix .. asn, sc)
+    end
+    if country then
+      rule.backend.set_token(task, rule, cfg.country_prefix .. country, sc)
+    end
 
-  rule.backend.set_token(task, rule, cfg.ip_prefix .. tostring(ip), sc)
+    rule.backend.set_token(task, rule, cfg.ip_prefix .. tostring(ip), sc)
+  end
 end
 
 -- Selectors are used to extract reputation tokens