]> source.dussan.org Git - rspamd.git/commitdiff
[Minor] Fix some more issues after actions rework
authorVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 14 Jan 2019 17:59:21 +0000 (17:59 +0000)
committerVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 14 Jan 2019 17:59:21 +0000 (17:59 +0000)
src/libmime/filter.c
src/libserver/task.c

index ea9b500c991a79e8670cec8c9f9ab8cd4a8e15e1..45563e23f11f469d22fea93655331eb3995b6d14 100644 (file)
@@ -494,7 +494,6 @@ rspamd_check_action_metric (struct rspamd_task *task)
        struct rspamd_passthrough_result *pr;
        double max_score = -(G_MAXDOUBLE), sc;
        int i;
-       gboolean set_action = FALSE;
        struct rspamd_metric_result *mres = task->result;
 
        /* We are not certain about the results during processing */
@@ -507,7 +506,8 @@ rspamd_check_action_metric (struct rspamd_task *task)
                                noaction = action_lim;
                        }
 
-                       if (isnan (sc)) {
+                       if (isnan (sc) ||
+                               (action_lim->action->flags & (RSPAMD_ACTION_NO_THRESHOLD|RSPAMD_ACTION_HAM))) {
                                continue;
                        }
 
index df1a561cbb2ab8a36b008c206f98772f6b046c08..2b00f68058c400f5cabcf1b4cc98a953e4a6f3f7 100644 (file)
@@ -1553,7 +1553,11 @@ rspamd_task_get_required_score (struct rspamd_task *task, struct rspamd_metric_r
        }
 
        for (i = m->nactions - 1; i >= 0; i --) {
-               if (!isnan (m->actions_limits[i].cur_limit)) {
+               struct rspamd_action_result *action_lim = &m->actions_limits[i];
+
+
+               if (!isnan (action_lim->cur_limit) &&
+                               !(action_lim->action->flags & (RSPAMD_ACTION_NO_THRESHOLD|RSPAMD_ACTION_HAM))) {
                        return m->actions_limits[i].cur_limit;
                }
        }