commit 4d52a58: [Fix] Dmarc: Always lowercase domain

Vsevolod Stakhov vsevolod at highsecure.ru
Mon Oct 18 09:56:04 UTC 2021


Author: Vsevolod Stakhov
Date: 2021-10-18 10:49:53 +0100
URL: https://github.com/rspamd/rspamd/commit/4d52a5896a9b795faffdd4142363a37582c8e346 (HEAD -> master)

[Fix] Dmarc: Always lowercase domain
Issue: #3940

---
 src/plugins/lua/dmarc.lua | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/plugins/lua/dmarc.lua b/src/plugins/lua/dmarc.lua
index ffc37ae3d..d5073377b 100644
--- a/src/plugins/lua/dmarc.lua
+++ b/src/plugins/lua/dmarc.lua
@@ -131,7 +131,7 @@ local function dmarc_validate_policy(task, policy, hdrfromdom, dmarc_esld)
 
     for _,opt in ipairs(opts) do
       local check_res = string.sub(opt, -1)
-      local domain = string.sub(opt, 1, -3)
+      local domain = string.sub(opt, 1, -3):lower()
 
       if check_res == '+' then
         table.insert(dkim_results.pass, domain)
@@ -357,6 +357,8 @@ local function dmarc_callback(task)
 
   -- Do some initial sanity checks, detect tld domain if different
   if hfromdom and hfromdom ~= '' and not (from or E)[2] then
+    -- Lowercase domain as per #3940
+    hfromdom = hfromdom:lower()
     dmarc_domain = rspamd_util.get_tld(hfromdom)
   elseif (from or E)[2] then
     task:insert_result(settings.symbols['na'], 1.0, 'Duplicate From header')


More information about the Commits mailing list