]> source.dussan.org Git - rspamd.git/commitdiff
Fix configtest
authorVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 30 Nov 2015 12:56:07 +0000 (12:56 +0000)
committerVsevolod Stakhov <vsevolod@highsecure.ru>
Mon, 30 Nov 2015 12:56:07 +0000 (12:56 +0000)
src/libserver/cfg_utils.c
src/rspamadm/configdump.c
src/rspamadm/configtest.c

index 4fdc78526797c56ee350988815831aa2bfe1ff71..9447eaf32fbe672017eb8d2b01fb6af0795e5b31 100644 (file)
@@ -658,7 +658,7 @@ rspamd_config_post_load (struct rspamd_config *cfg, gboolean validate_cache)
        rspamd_url_init (cfg->tld_file);
 
        /* Insert classifiers symbols */
-       (void)rspamd_config_insert_classify_symbols (cfg);
+       rspamd_config_insert_classify_symbols (cfg);
 
        /* Parse format string that we have */
        if (!rspamd_config_parse_log_format (cfg)) {
@@ -986,7 +986,7 @@ symbols_classifiers_callback (gpointer key, gpointer value, gpointer ud)
 
        /* Actually, statistics should act like any ordinary symbol */
        rspamd_symbols_cache_add_symbol (cfg->cache, key, 0, NULL, NULL,
-                       SYMBOL_TYPE_COMPOSITE, -1);
+                       SYMBOL_TYPE_CLASSIFIER, -1);
 }
 
 void
index 1e169f791a8ff90e75d0ef219e7ac3ccd14f8abf..fbff4b23b6c8c0d80d7756960afcbac3e59d97c4 100644 (file)
@@ -129,7 +129,7 @@ rspamadm_configdump (gint argc, gchar **argv)
        const gchar *confdir;
        const ucl_object_t *obj, *cur;
        struct rspamd_config *cfg = rspamd_main->cfg;
-       gboolean ret = FALSE;
+       gboolean ret = TRUE;
        worker_t **pworker;
        gchar **sec;
 
index 9c8353312d881eb0b5ab58577e90f5c7177d04c3..26bdd028678b0fcf11bb6249ee72f8c7b3a6d9fb 100644 (file)
@@ -106,7 +106,7 @@ rspamadm_configtest (gint argc, gchar **argv)
        GError *error = NULL;
        const gchar *confdir;
        struct rspamd_config *cfg = rspamd_main->cfg;
-       gboolean ret = FALSE;
+       gboolean ret = TRUE;
        worker_t **pworker;
        const guint64 *log_cnt;
 
@@ -156,15 +156,16 @@ rspamadm_configtest (gint argc, gchar **argv)
                if (!rspamd_init_filters (rspamd_main->cfg, FALSE)) {
                        ret = FALSE;
                }
+
+               if (ret) {
+                       ret = rspamd_config_post_load (cfg, FALSE);
+               }
+
                if (!rspamd_symbols_cache_validate (rspamd_main->cfg->cache,
                                rspamd_main->cfg,
                                FALSE)) {
                        ret = FALSE;
                }
-
-               if (ret) {
-                       ret = rspamd_config_post_load (cfg, FALSE);
-               }
        }
 
        if (strict && ret) {