diff options
author | João Valverde <joao.valverde@tecnico.ulisboa.pt> | 2015-08-15 08:29:51 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2015-08-15 15:20:13 +0000 |
commit | 85e2018eac2540214aeaca37549cbdf4fde6f47a (patch) | |
tree | 8674a3d3c18b316a3a94edee07881a3a8abb7fbd /epan | |
parent | 85d2eef6193e7516b25a1fa05e4e000774846455 (diff) |
udp: fix minimum jumbo length
Change-Id: I5ab4974cc60cdec0c3571bdec815c8cb9e8f7255
Reviewed-on: https://code.wireshark.org/review/10040
Reviewed-by: Michael Mann <mmann78@netscape.net>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-udp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-udp.c b/epan/dissectors/packet-udp.c index afc6cfebfd..ae795158ef 100644 --- a/epan/dissectors/packet-udp.c +++ b/epan/dissectors/packet-udp.c @@ -655,7 +655,7 @@ dissect(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, guint32 ip_proto) } else { if (tree) { ti = proto_tree_add_uint(udp_tree, &hfi_udp_length, tvb, offset + 4, 2, udph->uh_ulen); - if (udp_len_zero && tvb_reported_length(tvb) < 35635) { + if (udp_len_zero && (tvb_reported_length(tvb) < 35636)) { expert_add_info(pinfo, ti, &ei_udp_len_zero_bad); } /* XXX - why is this here, given that this is UDP, not Lightweight UDP? */ @@ -955,7 +955,7 @@ proto_register_udp(void) { &ei_udplite_checksum_coverage, { "udp.checksum_coverage.expert", PI_MALFORMED, PI_ERROR, "Bad checksum coverage length value", EXPFILL }}, { &ei_udp_checksum_zero, { "udp.checksum.zero", PI_CHECKSUM, PI_ERROR, "Illegal Checksum value (0)", EXPFILL }}, { &ei_udp_checksum_bad, { "udp.checksum_bad.expert", PI_CHECKSUM, PI_ERROR, "Bad checksum", EXPFILL }}, - { &ei_udp_len_zero_bad, { "udp.length_zero_bad", PI_PROTOCOL, PI_WARN, "UDP lenth is zero but payload < 65535", EXPFILL }}, + { &ei_udp_len_zero_bad, { "udp.length_zero_bad", PI_PROTOCOL, PI_WARN, "UDP Length is zero but payload < 65536", EXPFILL }}, }; static build_valid_func udp_da_src_values[1] = {udp_src_value}; |