diff options
Diffstat (limited to 'src/plugins')
-rw-r--r-- | src/plugins/chartable.c | 15 | ||||
-rw-r--r-- | src/plugins/dkim_check.c | 10 | ||||
-rw-r--r-- | src/plugins/fuzzy_check.c | 6 | ||||
-rw-r--r-- | src/plugins/regexp.c | 1 | ||||
-rw-r--r-- | src/plugins/spf.c | 1 | ||||
-rw-r--r-- | src/plugins/surbl.c | 42 |
6 files changed, 30 insertions, 45 deletions
diff --git a/src/plugins/chartable.c b/src/plugins/chartable.c index 937cf1dc0..e33052063 100644 --- a/src/plugins/chartable.c +++ b/src/plugins/chartable.c @@ -36,7 +36,6 @@ #include "message.h" #include "cfg_file.h" #include "expressions.h" -#include "view.h" #define DEFAULT_SYMBOL "R_CHARSET_MIXED" #define DEFAULT_THRESHOLD 0.1 @@ -202,15 +201,13 @@ chartable_symbol_callback (struct rspamd_task *task, void *unused) GList *cur; struct mime_text_part *part; - if (check_view (task->cfg->views, chartable_module_ctx->symbol, task)) { - cur = g_list_first (task->text_parts); - while (cur) { - part = cur->data; - if (!part->is_empty && check_part (part, task->cfg->raw_mode)) { - insert_result (task, chartable_module_ctx->symbol, 1, NULL); - } - cur = g_list_next (cur); + cur = g_list_first (task->text_parts); + while (cur) { + part = cur->data; + if (!part->is_empty && check_part (part, task->cfg->raw_mode)) { + insert_result (task, chartable_module_ctx->symbol, 1, NULL); } + cur = g_list_next (cur); } } diff --git a/src/plugins/dkim_check.c b/src/plugins/dkim_check.c index 9610db6a8..ac12c0909 100644 --- a/src/plugins/dkim_check.c +++ b/src/plugins/dkim_check.c @@ -43,7 +43,6 @@ #include "cfg_file.h" #include "expressions.h" #include "util.h" -#include "view.h" #include "map.h" #include "dkim.h" #include "hash.h" @@ -315,12 +314,9 @@ dkim_symbol_callback (struct rspamd_task *task, void *unused) if (hlist != NULL) { /* Check whitelist */ msg_debug ("dkim signature found"); -#ifdef HAVE_INET_PTON - if (!task->from_addr.has_addr || - radix32tree_find (dkim_module_ctx->whitelist_ip, ntohl (task->from_addr.d.in4.s_addr)) == RADIX_NO_VALUE) { -#else - if (radix32tree_find (dkim_module_ctx->whitelist_ip, ntohl (task->from_addr.s_addr)) == RADIX_NO_VALUE) { -#endif + if (task->from_addr.af == AF_INET || + radix32tree_find (dkim_module_ctx->whitelist_ip, + ntohl (task->from_addr.addr.s4.sin_addr.s_addr)) == RADIX_NO_VALUE) { /* Parse signature */ msg_debug ("create dkim signature"); /* diff --git a/src/plugins/fuzzy_check.c b/src/plugins/fuzzy_check.c index 595cfc99b..195ee19da 100644 --- a/src/plugins/fuzzy_check.c +++ b/src/plugins/fuzzy_check.c @@ -45,7 +45,6 @@ #include "cfg_file.h" #include "expressions.h" #include "util.h" -#include "view.h" #include "map.h" #include "images.h" #include "fuzzy_storage.h" @@ -823,8 +822,9 @@ fuzzy_symbol_callback (struct rspamd_task *task, void *unused) /* Check whitelist */ #ifdef HAVE_INET_PTON - if (fuzzy_module_ctx->whitelist && !task->from_addr.ipv6 && task->from_addr.d.in4.s_addr != INADDR_NONE) { - if (radix32tree_find (fuzzy_module_ctx->whitelist, ntohl ((guint32) task->from_addr.d.in4.s_addr)) != RADIX_NO_VALUE) { + if (fuzzy_module_ctx->whitelist && task->from_addr.af == AF_INET) { + if (radix32tree_find (fuzzy_module_ctx->whitelist, + ntohl (task->from_addr.addr.s4.sin_addr.s_addr)) != RADIX_NO_VALUE) { msg_info ("<%s>, address %s is whitelisted, skip fuzzy check", task->message_id, inet_ntoa (task->from_addr.d.in4)); return; diff --git a/src/plugins/regexp.c b/src/plugins/regexp.c index 0229236f6..ddc03db38 100644 --- a/src/plugins/regexp.c +++ b/src/plugins/regexp.c @@ -34,7 +34,6 @@ #include "map.h" #include "util.h" #include "expressions.h" -#include "view.h" #include "lua/lua_common.h" #include "json/jansson.h" diff --git a/src/plugins/spf.c b/src/plugins/spf.c index 149e3104d..94245ea7e 100644 --- a/src/plugins/spf.c +++ b/src/plugins/spf.c @@ -38,7 +38,6 @@ #include "cfg_file.h" #include "expressions.h" #include "util.h" -#include "view.h" #include "map.h" #include "spf.h" #include "hash.h" diff --git a/src/plugins/surbl.c b/src/plugins/surbl.c index 3358b65d0..59c13b450 100644 --- a/src/plugins/surbl.c +++ b/src/plugins/surbl.c @@ -47,7 +47,6 @@ #include "cfg_file.h" #include "expressions.h" #include "util.h" -#include "view.h" #include "map.h" #include "dns.h" #include "hash.h" @@ -634,31 +633,26 @@ make_surbl_requests (struct uri *url, struct rspamd_task *task, f.begin = url->host; f.len = url->hostlen; - if (check_view (task->cfg->views, suffix->symbol, task)) { - if ((surbl_req = format_surbl_request (task->task_pool, &f, suffix, TRUE, - &err, forced, tree)) != NULL) { - param = rspamd_mempool_alloc (task->task_pool, sizeof (struct dns_param)); - param->url = url; - param->task = task; - param->suffix = suffix; - param->host_resolve = rspamd_mempool_strdup (task->task_pool, surbl_req); - debug_task ("send surbl dns request %s", surbl_req); - if (make_dns_request (task->resolver, task->s, task->task_pool, dns_callback, - (void *)param, RDNS_REQUEST_A, surbl_req)) { - task->dns_requests ++; - } - } - else if (err != NULL && err->code != WHITELIST_ERROR && err->code != DUPLICATE_ERROR) { - msg_info ("cannot format url string for surbl %s, %s", struri (url), err->message); - g_error_free (err); - return; - } - else if (err != NULL) { - g_error_free (err); + if ((surbl_req = format_surbl_request (task->task_pool, &f, suffix, TRUE, + &err, forced, tree)) != NULL) { + param = rspamd_mempool_alloc (task->task_pool, sizeof (struct dns_param)); + param->url = url; + param->task = task; + param->suffix = suffix; + param->host_resolve = rspamd_mempool_strdup (task->task_pool, surbl_req); + debug_task ("send surbl dns request %s", surbl_req); + if (make_dns_request (task->resolver, task->s, task->task_pool, dns_callback, + (void *)param, RDNS_REQUEST_A, surbl_req)) { + task->dns_requests ++; } } - else { - debug_task ("skipping symbol that is not in view: %s", suffix->symbol); + else if (err != NULL && err->code != WHITELIST_ERROR && err->code != DUPLICATE_ERROR) { + msg_info ("cannot format url string for surbl %s, %s", struri (url), err->message); + g_error_free (err); + return; + } + else if (err != NULL) { + g_error_free (err); } } |