file detect: improve cleanup

src/detect-fileext.c

@@ -227,6 +227,8 @@ error:
 void DetectFileextFree(void *ptr) {
     if (ptr != NULL) {
         DetectFileextData *fileext = (DetectFileextData *)ptr;
+        if (fileext->ext != NULL)
+            SCFree(fileext->ext);
         SCFree(fileext);
     }
 }

src/detect-filemagic.c

@@ -297,6 +297,8 @@ void DetectFilemagicFree(void *ptr) {
         if (filemagic->bm_ctx != NULL) {
             BoyerMooreCtxDeInit(filemagic->bm_ctx);
         }
+        if (filemagic->name != NULL)
+            SCFree(filemagic->name);
         SCFree(filemagic);
     }
 }

src/detect-filename.c

@@ -238,6 +238,8 @@ void DetectFilenameFree(void *ptr) {
         if (filename->bm_ctx != NULL) {
             BoyerMooreCtxDeInit(filename->bm_ctx);
         }
+        if (filename->name != NULL)
+            SCFree(filename->name);
         SCFree(filename);
     }
 }

src/detect-filestore.c

@@ -59,6 +59,7 @@ static pcre_extra *parse_regex_study;
 
 int DetectFilestoreMatch (ThreadVars *, DetectEngineThreadCtx *, Flow *, uint8_t, void *, Signature *, SigMatch *);
 static int DetectFilestoreSetup (DetectEngineCtx *, Signature *, char *);
+static void DetectFilestoreFree(void *);
 
 /**
  * \brief Registration function for keyword: filestore
@@ -69,7 +70,7 @@ void DetectFilestoreRegister(void) {
     sigmatch_table[DETECT_FILESTORE].AppLayerMatch = DetectFilestoreMatch;
     sigmatch_table[DETECT_FILESTORE].alproto = ALPROTO_HTTP;
     sigmatch_table[DETECT_FILESTORE].Setup = DetectFilestoreSetup;
-    sigmatch_table[DETECT_FILESTORE].Free  = NULL;
+    sigmatch_table[DETECT_FILESTORE].Free  = DetectFilestoreFree;
     sigmatch_table[DETECT_FILESTORE].RegisterTests = NULL;
 
     const char *eb;
@@ -417,3 +418,9 @@ error:
         SCFree(sm);
     return -1;
 }
+
+static void DetectFilestoreFree(void *ptr) {
+    if (ptr != NULL) {
+        SCFree(ptr);
+    }
+}