]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
unix socket: fix valgrind issue
authorEric Leblond <eric@regit.org>
Fri, 20 Jun 2014 16:11:29 +0000 (18:11 +0200)
committerVictor Julien <victor@inliniac.net>
Mon, 23 Jun 2014 09:28:23 +0000 (11:28 +0200)
This patch fixes the following issue reported by valgrind:
 31 errors in context 1 of 1:
 Conditional jump or move depends on uninitialised value(s)
    at 0x8AB2F8: UnixSocketPcapFilesCheck (runmode-unix-socket.c:279)
    by 0x97725D: UnixCommandBackgroundTasks (unix-manager.c:368)
    by 0x97BC52: UnixManagerThread (unix-manager.c:884)
    by 0x6155F6D: start_thread (pthread_create.c:311)
    by 0x6E3A9CC: clone (clone.S:113)

The running field in PcapCommand was not initialized.

src/runmode-unix-socket.c

index 8a3d017f438e72cc7854645fea65ace8c0268a32..03d7790315e4dc26190ddc293915c99335a8a8e2 100644 (file)
@@ -382,6 +382,7 @@ int RunModeUnixSocketSingle(DetectEngineCtx *de_ctx)
     }
     pcapcmd->de_ctx = de_ctx;
     TAILQ_INIT(&pcapcmd->files);
+    pcapcmd->running = 0;
     pcapcmd->currentfile = NULL;
 
     UnixManagerThreadSpawn(de_ctx, 1);