]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
fuzz: rightly uses PacketFreeOrRelease in target 5896/head
authorPhilippe Antoine <contact@catenacyber.fr>
Wed, 27 Jan 2021 20:21:44 +0000 (21:21 +0100)
committerVictor Julien <victor@inliniac.net>
Mon, 22 Feb 2021 15:26:58 +0000 (16:26 +0100)
instead of PacketFree because packets
may belong to the pool

(cherry picked from commit 62e665c8482c90b30f6edfa7b0f0eabf8a4fcc79)

src/tests/fuzz/fuzz_sigpcap.c

index 26d8dcd29d95d4755382a27e101f46bc7a3b5643..2607f2d44ec43d7786ae85b2e5e88c7479ae96f9 100644 (file)
@@ -153,13 +153,13 @@ int LLVMFuzzerTestOneInput(const uint8_t *data, size_t size)
             }
             Packet *extra_p = PacketDequeueNoLock(&tv.decode_pq);
             while (extra_p != NULL) {
-                PacketFree(extra_p);
+                PacketFreeOrRelease(extra_p);
                 extra_p = PacketDequeueNoLock(&tv.decode_pq);
             }
             tmm_modules[TMM_FLOWWORKER].Func(&tv, p, fwd);
             extra_p = PacketDequeueNoLock(&tv.decode_pq);
             while (extra_p != NULL) {
-                PacketFree(extra_p);
+                PacketFreeOrRelease(extra_p);
                 extra_p = PacketDequeueNoLock(&tv.decode_pq);
             }
         }