From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Thu, 16 Mar 2017 18:23:31 +0000 (+0000)
Subject: [Fix] Fix URI_COUNT_ODD rule
X-Git-Tag: 1.5.3~11
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=9206ce91df6c796baf09b63963de7da668c25a63;p=rspamd.git

[Fix] Fix URI_COUNT_ODD rule
---

diff --git a/rules/misc.lua b/rules/misc.lua
index 2bba06e91..846da24d8 100644
--- a/rules/misc.lua
+++ b/rules/misc.lua
@@ -232,9 +232,13 @@ rspamd_config.URI_COUNT_ODD = {
   callback = function (task)
     local ct = task:get_header('Content-Type')
     if (ct and ct:lower():find('^multipart/alternative')) then
-      local urls = task:get_urls()
-      if (urls and (#urls % 2 == 1)) then
-        return true, 1.0, tostring(#urls % 2)
+      local urls = task:get_urls() or {}
+      local nurls = fun.filter(function(url)
+        return not url:is_html_displayed()
+      end, urls):foldl(function(acc, val) return acc + 1 end, 0)
+
+      if nurls % 2 == 1 then
+        return true, 1.0, tostring(nurls)
       end
     end
   end,