From 48ffe66c9aa764a691c416edb0f52a514397166b Mon Sep 17 00:00:00 2001 From: Vsevolod Stakhov Date: Tue, 5 Dec 2017 18:26:17 +0000 Subject: [PATCH] [Minor] Do not crash if no actions defined --- lualib/rspamd_config_transform.lua | 28 ++++++++++++++-------------- 1 file changed, 14 insertions(+), 14 deletions(-) diff --git a/lualib/rspamd_config_transform.lua b/lualib/rspamd_config_transform.lua index a46ede336c..e74c0f689c 100644 --- a/lualib/rspamd_config_transform.lua +++ b/lualib/rspamd_config_transform.lua @@ -256,21 +256,21 @@ return function(cfg) if not cfg.actions then logger.errx('no actions defined') - end - - -- Perform sanity check for actions - local actions_defs = {'greylist', 'add header', 'add_header', - 'rewrite subject', 'rewrite_subject', 'reject'} - - if not cfg.actions['no action'] and not cfg.actions['no_action'] and - not cfg.actions['accept'] then - for _,d in ipairs(actions_defs) do - if cfg.actions[d] then - if cfg.actions[d] < 0 then - cfg.actions['no action'] = cfg.actions[d] - 0.001 - logger.infox('set no action score to: %s, as action %s has negative score', + else + -- Perform sanity check for actions + local actions_defs = {'greylist', 'add header', 'add_header', + 'rewrite subject', 'rewrite_subject', 'reject'} + + if not cfg.actions['no action'] and not cfg.actions['no_action'] and + not cfg.actions['accept'] then + for _,d in ipairs(actions_defs) do + if cfg.actions[d] then + if cfg.actions[d] < 0 then + cfg.actions['no action'] = cfg.actions[d] - 0.001 + logger.infox('set no action score to: %s, as action %s has negative score', cfg.actions['no action'], d) - break + break + end end end end -- 2.47.3