]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
detect/csum: rm interaction btw stream setting/csum
authorJeff Lucovsky <jlucovsky@oisf.net>
Mon, 20 Jan 2025 13:46:39 +0000 (08:46 -0500)
committerVictor Julien <victor@inliniac.net>
Tue, 11 Feb 2025 09:54:15 +0000 (10:54 +0100)
Issue: 7467

Stream checksum validation no longer has a side effect of setting
PKT_IGNORE_CHECKSUM and thus, no longer affects csum keyword checks.

(cherry picked from commit 758da982f087dce249012304de7d3077adf9bade)

src/stream-tcp.c

index 46b379f7de163db932a93f9b65b3c26b4846ad3e..b847fb6f8a001380b97982ae7ce7acc2bc39de13 100644 (file)
@@ -5815,11 +5815,7 @@ TmEcode StreamTcp (ThreadVars *tv, Packet *p, void *data, PacketQueueNoLock *pq)
                 StatsIncr(tv, stt->counter_tcp_invalid_checksum);
                 return TM_ECODE_OK;
             }
-        } else {
-            p->flags |= PKT_IGNORE_CHECKSUM;
         }
-    } else {
-        p->flags |= PKT_IGNORE_CHECKSUM; //TODO check that this is set at creation
     }
     AppLayerProfilingReset(stt->ra_ctx->app_tctx);