diff options
author | Matthieu Coudron <mattator@gmail.com> | 2014-11-18 19:33:53 +0100 |
---|---|---|
committer | Evan Huus <eapache@gmail.com> | 2014-11-25 02:47:20 +0000 |
commit | 4f8a6ec8669fe0b9b2ac642ab99cc853498e4721 (patch) | |
tree | 730a13723ca11da4c3d569885113ef21903365fa /epan/dissectors/packet-tcp.h | |
parent | d5713e2040fc47a52651c09f1e373b531d3d370e (diff) |
TCP: fix wrongly set base_seq when ISN is 0 and tcp_relative_seq is
enabled
tcp_analysis::base_seq could be set several times when the
TCP ISN was set to 0, thus inducing some undesired wraps such as 0-1
Bug: 10713
Change-Id: I69a0dfe677e93bf51015bf7a39ebf888631b12a4
Reviewed-on: https://code.wireshark.org/review/5387
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-tcp.h')
-rw-r--r-- | epan/dissectors/packet-tcp.h | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/epan/dissectors/packet-tcp.h b/epan/dissectors/packet-tcp.h index b502527da6..f1b34738a1 100644 --- a/epan/dissectors/packet-tcp.h +++ b/epan/dissectors/packet-tcp.h @@ -151,9 +151,8 @@ struct tcp_multisegment_pdu { }; typedef struct _tcp_flow_t { - guint32 base_seq; /* base seq number (used by relative sequence numbers) - * or 0 if not yet known. - */ + gboolean base_seq_set; /* true if base seq set */ + guint32 base_seq; /* base seq number (used by relative sequence numbers)*/ tcp_unacked_t *segments; guint32 fin; /* frame number of the final FIN */ guint32 lastack; /* last seen ack */ |