]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
detect: consolidate more setup into DetectEngineCtxInit
authorVictor Julien <victor@inliniac.net>
Fri, 16 Jan 2015 18:28:44 +0000 (19:28 +0100)
committerVictor Julien <victor@inliniac.net>
Thu, 19 Mar 2015 08:03:51 +0000 (09:03 +0100)
Loading of classifications, references and action order was done
unconditionally, so can be done in one place.

src/detect-engine.c
src/suricata.c

index 248927822a1790b733eaa26abd9b6b87d888b406..9b2058ddb60a6d6019b7de25ab18fbeb7aabc7cc 100644 (file)
@@ -551,13 +551,6 @@ static void *DetectEngineLiveRuleSwap(void *arg)
         goto error;
     }
 
-    SCClassConfLoadClassficationConfigFile(de_ctx);
-    SCRConfLoadReferenceConfigFile(de_ctx);
-
-    if (ActionInitConfig() < 0) {
-        exit(EXIT_FAILURE);
-    }
-
     if (SigLoadSignatures(de_ctx, NULL, FALSE) < 0) {
         SCLogError(SC_ERR_NO_RULES_LOADED, "Loading signatures failed.");
         if (de_ctx->failure_fatal)
@@ -879,6 +872,13 @@ DetectEngineCtx *DetectEngineCtxInit(void)
     SCProfilingKeywordInitCounters(de_ctx);
 #endif
 
+    SCClassConfLoadClassficationConfigFile(de_ctx);
+    SCRConfLoadReferenceConfigFile(de_ctx);
+
+    if (ActionInitConfig() < 0) {
+        goto error;
+    }
+
     return de_ctx;
 error:
     return NULL;
index d385b5b381f209d35cd90e3997c83152c4207217..12458caf70e84b7f5467debdfb789974b473dc4d 100644 (file)
@@ -2312,12 +2312,6 @@ int main(int argc, char **argv)
             CudaVarsSetDeCtx(de_ctx);
 #endif /* __SC_CUDA_SUPPORT__ */
 
-        SCClassConfLoadClassficationConfigFile(de_ctx);
-        SCRConfLoadReferenceConfigFile(de_ctx);
-
-        if (ActionInitConfig() < 0) {
-            exit(EXIT_FAILURE);
-        }
     } else {
         /* disable raw reassembly */
         (void)ConfSetFinal("stream.reassembly.raw", "false");