From baffa62e42485b362aee4472e85e9e676aba5b1d Mon Sep 17 00:00:00 2001 From: Vsevolod Stakhov Date: Sun, 7 Aug 2016 23:45:13 +0100 Subject: [PATCH] [Fix] Fix dependencies id sanity check --- src/libserver/symbols_cache.c | 15 ++------------- 1 file changed, 2 insertions(+), 13 deletions(-) diff --git a/src/libserver/symbols_cache.c b/src/libserver/symbols_cache.c index 02f45f522..bc91dce24 100644 --- a/src/libserver/symbols_cache.c +++ b/src/libserver/symbols_cache.c @@ -1238,15 +1238,8 @@ rspamd_symbols_cache_check_deps (struct rspamd_task *task, if (dep->item == NULL) { /* Assume invalid deps as done */ - continue; - } - - if (dep->id >= (gint)checkpoint->version) { - /* - * This is dependency on some symbol that is currently - * not in this checkpoint. So we just pretend that the - * corresponding symbold does not exist - */ + msg_warn_task ("symbol %s has invalid dependencies from %s", + item->symbol, dep->sym); continue; } @@ -1577,10 +1570,6 @@ rspamd_symbols_cache_process_symbols (struct rspamd_task * task, for (i = 0; i < (gint)checkpoint->waitq->len; i ++) { item = g_ptr_array_index (checkpoint->waitq, i); - if (item->id >= (gint)checkpoint->version) { - continue; - } - if (!isset (checkpoint->processed_bits, item->id * 2)) { if (!rspamd_symbols_cache_check_deps (task, cache, item, checkpoint, 0)) { -- 2.39.5