diff options
author | Stig Bjørlykke <stig@bjorlykke.org> | 2015-11-13 15:34:56 +0100 |
---|---|---|
committer | Stig Bjørlykke <stig@bjorlykke.org> | 2015-11-15 18:25:50 +0000 |
commit | 5d41bb3f1837cf41fc078050845c777cfda92bee (patch) | |
tree | f37d135457c129a3a70f39e86d4aa6631d4d02ed /epan | |
parent | df05544da6b833c4a304510e29a496f28cadbb00 (diff) | |
download | wireshark-5d41bb3f1837cf41fc078050845c777cfda92bee.tar.gz |
udp: Don't mark zero checksum illegal when in_error_pkt.
A zero checksum is not illegal in IPv6/UDP when in a ICMPv6 packet.
Change-Id: I07acc874d2385992089ef3ebc7a82e853904ecfc
Ping-Bug: 6232
Reviewed-on: https://code.wireshark.org/review/11808
Reviewed-by: Stig Bjørlykke <stig@bjorlykke.org>
(cherry picked from commit 82bd3cc9eb49c886a1b6ff1ad0aa3be096bd8aec)
Reviewed-on: https://code.wireshark.org/review/11843
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-udp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-udp.c b/epan/dissectors/packet-udp.c index 9d3ed4c031..29bc26ea23 100644 --- a/epan/dissectors/packet-udp.c +++ b/epan/dissectors/packet-udp.c @@ -812,7 +812,7 @@ dissect(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, guint32 ip_proto) if (udph->uh_sum == 0) { /* No checksum supplied in the packet. */ - if ((ip_proto == IP_PROTO_UDP) && (pinfo->src.type == AT_IPv4)) { + if (((ip_proto == IP_PROTO_UDP) && (pinfo->src.type == AT_IPv4)) || pinfo->flags.in_error_pkt) { item = proto_tree_add_uint_format_value(udp_tree, hfi_udp_checksum.id, tvb, offset + 6, 2, 0, "0x%04x (none)", 0); |