]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
packetpool: remove debug validation check
authorVictor Julien <vjulien@oisf.net>
Thu, 13 Oct 2022 17:04:46 +0000 (19:04 +0200)
committerVictor Julien <vjulien@oisf.net>
Wed, 19 Oct 2022 21:03:24 +0000 (23:03 +0200)
Current packet might be the root or a child. Root would have set
drop action set, but Packet::pkt_src might be set in either.

src/tmqh-packetpool.c

index 5c78968e61f9d8cb0b10fb17a9b394c545d859df..f715c393f840785b08e3f22e0aeee9989d9e90d6 100644 (file)
@@ -363,14 +363,6 @@ void TmqhOutputPacketpool(ThreadVars *t, Packet *p)
     SCEnter();
     SCLogDebug("Packet %p, p->root %p, alloced %s", p, p->root, p->flags & PKT_ALLOC ? "true" : "false");
 
-#ifdef DEBUG_VALIDATION
-    /* Check that the drop reason has been set, if we have a drop.
-     * Use macro to avoid testing Packet::root. */
-    if (PACKET_TEST_ACTION(p, ACTION_DROP)) {
-        DEBUG_VALIDATE_BUG_ON((p)->drop_reason == PKT_DROP_REASON_NOT_SET);
-    }
-#endif
-
     if (IS_TUNNEL_PKT(p)) {
         SCLogDebug("Packet %p is a tunnel packet: %s",
             p,p->root ? "upper layer" : "tunnel root");