]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
pcap: fix minor scan-build warnings 5361/head
authorVictor Julien <victor@inliniac.net>
Fri, 4 Sep 2020 08:52:04 +0000 (10:52 +0200)
committerVictor Julien <victor@inliniac.net>
Fri, 4 Sep 2020 11:15:01 +0000 (13:15 +0200)
src/source-pcap-file.c
src/source-pcap.c

index 6d2c0b564aa3f1d7b56b84663298235156650cd8..4b2c28550d423c94ddb8596baf5d10eb3baaa77e 100644 (file)
@@ -307,6 +307,8 @@ TmEcode ReceivePcapFileThreadInit(ThreadVars *tv, const void *initdata, void **d
         if (pv->should_recurse == true && pv->should_loop == true) {
             SCLogError(SC_ERR_INVALID_ARGUMENT, "Error, --pcap-file-continuous and --pcap-file-recursive "
                                                 "cannot be used together.");
+            CleanupPcapFileDirectoryVars(pv);
+            CleanupPcapFileThreadVars(ptv);
             SCReturnInt(TM_ECODE_FAILED);
         }
 
index 63cc425d17f1f118a548596a5e37953339bd0282..9e49b40e178edb99ba0c51f8955bb5279dd3c964 100644 (file)
@@ -653,14 +653,13 @@ void PcapTranslateIPToDevice(char *pcap_dev, size_t len)
 
     struct addrinfo ai_hints;
     struct addrinfo *ai_list = NULL;
-    int ret = 0;
 
     memset(&ai_hints, 0, sizeof(ai_hints));
     ai_hints.ai_family = AF_UNSPEC;
     ai_hints.ai_flags = AI_NUMERICHOST;
 
     /* try to translate IP */
-    if ((ret = getaddrinfo(pcap_dev, NULL, &ai_hints, &ai_list)) != 0) {
+    if (getaddrinfo(pcap_dev, NULL, &ai_hints, &ai_list) != 0) {
         return;
     }