From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Thu, 13 May 2021 20:36:05 +0000 (+0100)
Subject: [Minor] Fix bit operations logic
X-Git-Tag: 3.0~405
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0b88d35b45dcea804e47a9236a5e0244b9b70c4e;p=rspamd.git

[Minor] Fix bit operations logic
---

diff --git a/rules/misc.lua b/rules/misc.lua
index a7e77516d..2e5f38800 100644
--- a/rules/misc.lua
+++ b/rules/misc.lua
@@ -127,16 +127,17 @@ rspamd_config:register_symbol({
 local obscured_id = rspamd_config:register_symbol{
   callback = function(task)
     local susp_urls = task:get_urls_filtered({ 'obscured', 'zw_spaces'})
-    local obs_flag = url_flags_tab.obscured
-    local zw_flag = url_flags_tab.zw_spaces
 
-    if susp_urls then
+    if susp_urls and susp_urls[1] then
+      local obs_flag = url_flags_tab.obscured
+      local zw_flag = url_flags_tab.zw_spaces
+
       for _,u in ipairs(susp_urls) do
         local fl = u:get_flags_num()
-        if bit.band(fl, obs_flag) then
+        if bit.band(fl, obs_flag) ~= 0 then
           task:insert_result('R_SUSPICIOUS_URL', 1.0, u:get_host())
         end
-        if bit.band(fl, zw_flag) then
+        if bit.band(fl, zw_flag) ~= 0 then
           task:insert_result('ZERO_WIDTH_SPACE_URL', 1.0, u:get_host())
         end
       end