]> git.ipfire.org Git - thirdparty/suricata.git/commitdiff
stream/reassemble: remove unused min_inspect_depth
authorShivani Bhardwaj <shivani@oisf.net>
Fri, 9 May 2025 06:56:40 +0000 (12:26 +0530)
committerVictor Julien <victor@inliniac.net>
Sat, 10 May 2025 01:09:50 +0000 (03:09 +0200)
Whether minimum inspection depth should be respected is an important
parameter but it is only used by one caller of StreamReassembleRawDo fn
to calculate progress value. This progress value is already passed as a
parameter to the said fn.

src/stream-tcp-reassemble.c

index 2c5f53bcfa265961c7373dd5445c7b2e5583bae5..a1cd39563df17dec49398595e8a5d8ad09e0c7a5 100644 (file)
@@ -1804,14 +1804,10 @@ static int StreamReassembleRawInline(TcpSession *ssn, const Packet *p,
  *  \param[out] progress_out absolute progress value of the data this
  *                           call handled.
  *  \param eof we're wrapping up so inspect all data we have, incl unACKd
- *  \param respect_inspect_depth use Stream::min_inspect_depth if set
- *
- *  `respect_inspect_depth` is used to avoid useless inspection of too
- *  much data.
  */
 static int StreamReassembleRawDo(const TcpSession *ssn, const TcpStream *stream,
         StreamReassembleRawFunc Callback, void *cb_data, const uint64_t progress_in,
-        const uint64_t re, uint64_t *progress_out, bool eof, bool respect_inspect_depth)
+        const uint64_t re, uint64_t *progress_out, bool eof)
 {
     SCEnter();
     int r = 0;
@@ -1903,7 +1899,7 @@ int StreamReassembleForFrame(TcpSession *ssn, TcpStream *stream, StreamReassembl
 
     uint64_t unused = 0;
     return StreamReassembleRawDo(
-            ssn, stream, Callback, cb_data, offset, app_progress, &unused, eof, false);
+            ssn, stream, Callback, cb_data, offset, app_progress, &unused, eof);
 }
 
 int StreamReassembleRaw(TcpSession *ssn, const Packet *p,
@@ -1968,7 +1964,7 @@ int StreamReassembleRaw(TcpSession *ssn, const Packet *p,
     SCLogDebug("last_ack_abs %" PRIu64, last_ack_abs);
 
     return StreamReassembleRawDo(ssn, stream, Callback, cb_data, progress, last_ack_abs,
-            progress_out, (p->flags & PKT_PSEUDO_STREAM_END), respect_inspect_depth);
+            progress_out, (p->flags & PKT_PSEUDO_STREAM_END));
 }
 
 int StreamReassembleLog(const TcpSession *ssn, const TcpStream *stream,
@@ -1983,7 +1979,7 @@ int StreamReassembleLog(const TcpSession *ssn, const TcpStream *stream,
     SCLogDebug("last_ack_abs %" PRIu64, last_ack_abs);
 
     return StreamReassembleRawDo(
-            ssn, stream, Callback, cb_data, progress_in, last_ack_abs, progress_out, eof, false);
+            ssn, stream, Callback, cb_data, progress_in, last_ack_abs, progress_out, eof);
 }
 
 /** \internal