fixes for init failure stuff

src/detect-parse.c

@@ -663,13 +663,12 @@ Signature *SigInit(DetectEngineCtx *de_ctx, char *sigstr) {
     return sig;
 
 error:
-    SigFree(sig);
+    if ( sig != NULL ) SigFree(sig);
     if (de_ctx->failure_fatal == 1) {
-        fprintf(stderr, "ERROR: Signature init failed.\n");
+        SCLogError(SC_ERR_INVALID_SIGNATURE,"Signature init failed %s ",sigstr);
         exit(EXIT_FAILURE);
     }
     return NULL;
-
 }
 
 /**
@@ -846,6 +845,10 @@ Signature *DetectEngineAppendSig(DetectEngineCtx *de_ctx, char *sigstr) {
 
 error:
     if ( sig != NULL ) SigFree(sig);
+    if (de_ctx->failure_fatal == 1) {
+        SCLogError(SC_ERR_INVALID_SIGNATURE,"Signature init failed %s ",sigstr);
+        exit(EXIT_FAILURE);
+    }
     return NULL;
 }
 

src/suricata.c

@@ -273,9 +273,9 @@ void usage(const char *progname)
     printf("\t-u                           : run the unittests and exit\n");
     printf("\t-U, --unittest-filter=REGEX  : filter unittests with a regex\n");
     printf("\t--list-unittests             : list unit tests\n");
-    printf("\t--init-errors-fatal          : enable fatal failure on signature init error\n");
     printf("\t--fatal-unittests            : enable fatal failure on unittest error\n");
 #endif /* UNITTESTS */
+    printf("\t--init-errors-fatal          : enable fatal failure on signature init error\n");
     printf("\n");
     printf("\nTo run the engine with default configuration on "
             "interface eth0 with signature file \"signatures.rules\", run the "

src/util-error.c

@@ -25,6 +25,7 @@ const char * SCErrorToString(SCError err)
         CASE_CODE (SC_PCRE_PARSE_FAILED);
         CASE_CODE (SC_LOG_MODULE_NOT_INIT);
         CASE_CODE (SC_LOG_FG_FILTER_MATCH_FAILED);
+        CASE_CODE (SC_ERR_INVALID_SIGNATURE);
         CASE_CODE (SC_ERR_OPENING_FILE);
         CASE_CODE (SC_ERR_OPENING_RULE_FILE);
         CASE_CODE (SC_ERR_NO_RULES);