From: Michael Matirko (mmatirko) <mmatirko@cisco.com>
Date: Tue, 21 May 2024 15:22:42 +0000 (+0000)
Subject: Pull request #4328: flow: add the newly-created flow to p->flow to avoid segv
X-Git-Tag: 3.2.2.0~13
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=caccd0b8d89ce90059b97f047ebba71571cfbda0;p=thirdparty%2Fsnort3.git

Pull request #4328: flow: add the newly-created flow to p->flow to avoid segv

Merge in SNORT/snort3 from ~MMATIRKO/snort3:flowip_segv to master

Squashed commit of the following:

commit a24e308afb9eaef51ccad16a2a269918fdf607c0
Author: Michael Matirko <mmatirko@cisco.com>
Date:   Wed May 8 20:05:48 2024 -0400

    flow: add the newly-created flow to p->flow to avoid segv
---

diff --git a/src/flow/ha.cc b/src/flow/ha.cc
index 978ac618d..b08541a11 100644
--- a/src/flow/ha.cc
+++ b/src/flow/ha.cc
@@ -438,6 +438,7 @@ static Flow* consume_ha_update_message(HAMessage& msg, const FlowKey& key, Packe
 
     if( p && no_flow_found && flow && flow->session )
     {
+        p->flow = flow;
         flow->session->setup(p);
         flow->set_direction(p);
         flow->set_client_initiate(p);
diff --git a/src/network_inspectors/perf_monitor/perf_monitor.cc b/src/network_inspectors/perf_monitor/perf_monitor.cc
index 59a08f439..3a7720802 100644
--- a/src/network_inspectors/perf_monitor/perf_monitor.cc
+++ b/src/network_inspectors/perf_monitor/perf_monitor.cc
@@ -94,7 +94,7 @@ public:
     {
         FlowIPTracker* tracker = perf_monitor.get_flow_ip();
 
-        if (!tracker)
+        if (!tracker or !flow)
             return;
 
         FlowState state = SFS_STATE_MAX;