From: Vsevolod Stakhov <vsevolod@highsecure.ru>
Date: Sun, 7 Aug 2016 22:45:13 +0000 (+0100)
Subject: [Fix] Fix dependencies id sanity check
X-Git-Tag: 1.3.2~10
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=baffa62e42485b362aee4472e85e9e676aba5b1d;p=thirdparty%2Frspamd.git

[Fix] Fix dependencies id sanity check
---

diff --git a/src/libserver/symbols_cache.c b/src/libserver/symbols_cache.c
index 02f45f522e..bc91dce24d 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)) {