]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
drop-json: fix cleanup
authorVictor Julien <victor@inliniac.net>
Thu, 10 Apr 2014 07:40:42 +0000 (09:40 +0200)
committerVictor Julien <victor@inliniac.net>
Thu, 10 Apr 2014 07:40:42 +0000 (09:40 +0200)
Use proper function for sub-module cleanup. LogFileCtx is not managed
by the sub-module, so don't clean it.

src/output-json-drop.c

index a9cb069e1310bd3042edc32635bdaac9c5ca95fa..6be56f3e6b61bf20de7f62458e52c970b8367f7f 100644 (file)
@@ -193,6 +193,14 @@ static void JsonDropLogDeInitCtx(OutputCtx *output_ctx)
     SCFree(output_ctx);
 }
 
+static void JsonDropLogDeInitCtxSub(OutputCtx *output_ctx)
+{
+    OutputDropLoggerDisable();
+
+    SCLogDebug("cleaning up sub output_ctx %p", output_ctx);
+    SCFree(output_ctx);
+}
+
 #define DEFAULT_LOG_FILENAME "drop.json"
 static OutputCtx *JsonDropLogInitCtx(ConfNode *conf)
 {
@@ -238,7 +246,7 @@ static OutputCtx *JsonDropLogInitCtxSub(ConfNode *conf, OutputCtx *parent_ctx)
     }
 
     output_ctx->data = ajt->file_ctx;
-    output_ctx->DeInit = JsonDropLogDeInitCtx;
+    output_ctx->DeInit = JsonDropLogDeInitCtxSub;
     return output_ctx;
 }