diff options
| author | John Thacker <johnthacker@gmail.com> | 2022-08-26 00:17:40 -0400 |
|---|---|---|
| committer | A Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2022-08-27 11:47:42 +0000 |
| commit | e725502afae8da3e614b90429703e73f713fd58c (patch) | |
| tree | 50e65795a4e9a9376628103806c184a4fe4251ec /epan/dissectors/packet-tcp.c | |
| parent | 89457e01dac84e4dc5483db1539747d3df927d06 (diff) | |
tcp: Fix a comparison that should be strict in OOO processing
We don't want to add a zero length segment to the reassembly here.
Related to #17406
Diffstat (limited to 'epan/dissectors/packet-tcp.c')
| -rw-r--r-- | epan/dissectors/packet-tcp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-tcp.c b/epan/dissectors/packet-tcp.c index 53a01dce4c..8b95c6695e 100644 --- a/epan/dissectors/packet-tcp.c +++ b/epan/dissectors/packet-tcp.c @@ -3674,7 +3674,7 @@ split_msp(packet_info *pinfo, struct tcp_multisegment_pdu *msp, struct tcp_analy guint32 frag_offset = fd_i->offset; guint32 frag_len = fd_i->len; /* Check for some unusual out of order overlapping segment situations. */ - if (split_offset <= frag_offset + frag_len) { + if (split_offset < frag_offset + frag_len) { if (fd_i->offset < split_offset) { frag_offset = split_offset; frag_len -= (split_offset - fd_i->offset); |