From: Vsevolod Stakhov <vsevolod@rspamd.com>
Date: Thu, 1 May 2025 13:47:44 +0000 (+0100)
Subject: [Fix] Add fail check for cfg transform for some corner cases
X-Git-Tag: 3.12.0~35
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=eb7dd8bed18e4254e91985344e3287497619991b;p=thirdparty%2Frspamd.git

[Fix] Add fail check for cfg transform for some corner cases
---

diff --git a/lualib/lua_cfg_transform.lua b/lualib/lua_cfg_transform.lua
index 265ca34c0d..aaa0c7390d 100644
--- a/lualib/lua_cfg_transform.lua
+++ b/lualib/lua_cfg_transform.lua
@@ -198,20 +198,22 @@ end
 
 local function symbol_transform(cfg, k, v)
   local groups = cfg:at('group')
-  -- first try to find any group where there is a definition of this symbol
-  for gr_n, gr in groups:pairs() do
-    local symbols = gr:at('symbols')
-    if symbols and symbols:at(k) then
-      -- We override group symbol with ungrouped symbol
-      logger.infox("overriding group symbol %s in the group %s", k, gr_n)
-      symbols[k] = lua_util.override_defaults(symbols:at(k):unwrap(), v:unwrap())
-      return
+  if groups then
+    -- first try to find any group where there is a definition of this symbol
+    for gr_n, gr in groups:pairs() do
+      local symbols = gr:at('symbols')
+      if symbols and symbols:at(k) then
+        -- We override group symbol with ungrouped symbol
+        logger.infox("overriding group symbol %s in the group %s", k, gr_n)
+        symbols[k] = lua_util.override_defaults(symbols:at(k):unwrap(), v:unwrap())
+        return
+      end
     end
   end
   -- Now check what Rspamd knows about this symbol
   local sym = rspamd_config:get_symbol(k)
 
-  if not sym or not sym.group then
+  if groups and (not sym or not sym.group) then
     -- Otherwise we just use group 'ungrouped'
     if not groups:at('ungrouped') then
       groups.ungrouped = {