From 9b13c1b804f6411753e4b86b05226fc82d44691a Mon Sep 17 00:00:00 2001
From: Victor Julien <victor@inliniac.net>
Date: Thu, 30 Jul 2020 13:22:15 +0200
Subject: [PATCH] flow: avoid double state update on reuse

Avoids an unnecessary atomic operation.
---
 src/flow-hash.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/flow-hash.c b/src/flow-hash.c
index 17fec620ca..7c590016c3 100644
--- a/src/flow-hash.c
+++ b/src/flow-hash.c
@@ -574,6 +574,7 @@ static Flow *TcpReuseReplace(ThreadVars *tv, DecodeThreadVars *dtv,
     FlowInit(f, p);
     f->flow_hash = hash;
     f->fb = fb;
+    FlowUpdateState(f, FLOW_STATE_NEW);
 
     f->thread_id[0] = thread_id[0];
     f->thread_id[1] = thread_id[1];
@@ -952,8 +953,6 @@ static Flow *FlowGetUsedFlow(ThreadVars *tv, DecodeThreadVars *dtv)
 
         FlowClearMemory(f, f->protomap);
 
-        FlowUpdateState(f, FLOW_STATE_NEW);
-
         FLOWLOCK_UNLOCK(f);
         return f;
     }