diff options
| author | Michael Mann <mmann78@netscape.net> | 2017-06-04 17:35:15 -0400 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2017-06-05 02:53:35 +0000 |
| commit | d73fdc7079df7875e51d88cf763c0453e8f7c1a8 (patch) | |
| tree | eeee911e7894e5cd4173b31274ce2fe25f59eecb /epan/dissectors/packet-hartip.c | |
| parent | aabb8fd5946110767479e2863ffe4fc40d63ac53 (diff) | |
Remove "length only" check for dissectors that use tcp_dissect_pdus.
If a TCP segment is small enough, dissectors that have a only a length
check determining if it's their packet or not before calling tcp_dissect_pdus
will throw out packets that are probably destined for them.
Change-Id: I78034307b56aa537943191a6887166577936a6a3
Reviewed-on: https://code.wireshark.org/review/21950
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan/dissectors/packet-hartip.c')
| -rw-r--r-- | epan/dissectors/packet-hartip.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/epan/dissectors/packet-hartip.c b/epan/dissectors/packet-hartip.c index 79bedc76f0..9aa9665ed9 100644 --- a/epan/dissectors/packet-hartip.c +++ b/epan/dissectors/packet-hartip.c @@ -1385,9 +1385,6 @@ static int dissect_hartip_tcp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void *data) { - if (!tvb_bytes_exist(tvb, 0, HARTIP_HEADER_LENGTH)) - return 0; - tcp_dissect_pdus(tvb, pinfo, tree, hartip_desegment, HARTIP_HEADER_LENGTH, get_dissect_hartip_len, dissect_hartip_pdu, data); return tvb_reported_length(tvb); |