Pull request #4352: stream_tcp: set daq_msg field in meta-ack pseudo-packet header...

Merge in SNORT/snort3 from ~DAVMCPHE/snort3:meta_ack_daq_msg_init to master

Squashed commit of the following:

commit 63151959aaf66c274d277bd394b735f47007837e
Author: davis mcpherson <davmcphe@cisco.com>
Date:   Mon Jun 17 13:34:49 2024 -0400

    stream_tcp: set daq_msg field in meta-ack pseudo-packet header to the value from the wire packet.

diff --git a/src/stream/tcp/tcp_segment_descriptor.cc b/src/stream/tcp/tcp_segment_descriptor.cc
index 23fc6c23e2c77cb82da0c7eb297e0ad3368d6319..5613d0808476fcf3f214fbfd9ef5e30e2d286b0d 100644 (file)
--- a/src/stream/tcp/tcp_segment_descriptor.cc
+++ b/src/stream/tcp/tcp_segment_descriptor.cc
@@ -92,6 +92,7 @@ TcpSegmentDescriptor::TcpSegmentDescriptor
     pkt->flow = p->flow;
     pkt->context = p->context;
     pkt->dsize = 0;
+    pkt->daq_msg = p->daq_msg;
 
     seq = tcph->seq();
     ack = tcph->ack();

diff --git a/src/stream/tcp/tcp_session.cc b/src/stream/tcp/tcp_session.cc
index 6c0efd45d9e53e1704c6265145643ddaaa04a740..aa615790aa182abd2bf33a55de78d2bcb76fa4cf 100644 (file)
--- a/src/stream/tcp/tcp_session.cc
+++ b/src/stream/tcp/tcp_session.cc
@@ -1165,6 +1165,8 @@ int TcpSession::process(Packet* p)
     if ( tcp_mack )
     {
         TcpSegmentDescriptor ma_tsd(flow, p, tcp_mack->tcp_ack_seq_num, tcp_mack->tcp_window_size);
+        assert( ma_tsd.get_pkt()->daq_msg && ma_tsd.get_pkt()->daq_msg == p->daq_msg );
+
         init_tcp_packet_analysis(ma_tsd);
         process_tcp_packet(ma_tsd, p);
         tcpStats.meta_acks++;