flow: minor code cleanup

diff --git a/src/flow-manager.c b/src/flow-manager.c
index a06f8caa23d4f1b4f20c7fafe1fdcffad0afa4b6..7bffebeedcd5680a77c9d6e5dfffaddee00a1902 100644 (file)
--- a/src/flow-manager.c
+++ b/src/flow-manager.c
@@ -332,7 +332,7 @@ static uint32_t ProcessAsideQueue(FlowManagerTimeoutThread *td, FlowTimeoutCount
                 f->flow_state != FLOW_STATE_LOCAL_BYPASSED &&
                 FlowForceReassemblyNeedReassembly(f) == 1)
         {
-            FlowForceReassemblyForFlow(f); // TODO error check?
+            FlowForceReassemblyForFlow(f);
             /* flow ownership is passed to the worker thread */
 
             /* flow remains locked */

diff --git a/src/flow-timeout.c b/src/flow-timeout.c
index e48e7051220d30446c144104fb77bcf9745f1985..d5deb190d0831effbf05f402423f21280400d23e 100644 (file)
--- a/src/flow-timeout.c
+++ b/src/flow-timeout.c
@@ -345,11 +345,10 @@ int FlowForceReassemblyNeedReassembly(Flow *f)
  *
  * \retval 0 This flow doesn't need any reassembly processing; 1 otherwise.
  */
-int FlowForceReassemblyForFlow(Flow *f)
+void FlowForceReassemblyForFlow(Flow *f)
 {
     const int thread_id = (int)f->thread_id[0];
     TmThreadsInjectFlowById(f, thread_id);
-    return 1;
 }
 
 /**

diff --git a/src/flow-timeout.h b/src/flow-timeout.h
index 1430119caec3648f17b92c1d1b473d32932708d6..f6c95298799ba70ab2a400923dec35f20c55cef9 100644 (file)
--- a/src/flow-timeout.h
+++ b/src/flow-timeout.h
@@ -24,7 +24,7 @@
 #ifndef __FLOW_TIMEOUT_H__
 #define __FLOW_TIMEOUT_H__
 
-int FlowForceReassemblyForFlow(Flow *f);
+void FlowForceReassemblyForFlow(Flow *f);
 int FlowForceReassemblyNeedReassembly(Flow *f);
 void FlowForceReassembly(void);
 void FlowForceReassemblySetup(int detect_disabled);