diff options
author | Ulf Lamping <ulf.lamping@web.de> | 2005-07-08 22:11:13 +0000 |
---|---|---|
committer | Ulf Lamping <ulf.lamping@web.de> | 2005-07-08 22:11:13 +0000 |
commit | 2c30357c483da1238bfd45455272db93a2ec65c9 (patch) | |
tree | c7f025f042af2c3d960b0a9194442ab8ac8e93f0 /epan/dissectors/packet-ip.c | |
parent | 6b0815ee9cf23c43525c26814bf81a934e0120f2 (diff) |
checksum messages: use "[correct]" instead of "(correct)" and other "alike" messages
svn path=/trunk/; revision=14880
Diffstat (limited to 'epan/dissectors/packet-ip.c')
-rw-r--r-- | epan/dissectors/packet-ip.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/epan/dissectors/packet-ip.c b/epan/dissectors/packet-ip.c index 53d59f5b4f..ae62651662 100644 --- a/epan/dissectors/packet-ip.c +++ b/epan/dissectors/packet-ip.c @@ -1004,11 +1004,12 @@ dissect_ip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree) "Header checksum: 0x%04x [correct]", iph->ip_sum); } else { - item = proto_tree_add_boolean_hidden(ip_tree, hf_ip_checksum_bad, tvb, offset + 10, 2, TRUE); - PROTO_ITEM_SET_HIDDEN(item); proto_tree_add_uint_format(ip_tree, hf_ip_checksum, tvb, offset + 10, 2, iph->ip_sum, "Header checksum: 0x%04x [incorrect, should be 0x%04x]", iph->ip_sum, in_cksum_shouldbe(iph->ip_sum, ipsum)); + item = proto_tree_add_boolean(ip_tree, hf_ip_checksum_bad, tvb, offset + 10, 2, TRUE); + PROTO_ITEM_SET_GENERATED(item); + PROTO_ITEM_SET_HIDDEN(item); } } } else { @@ -1392,7 +1393,7 @@ dissect_mpls_extensions(tvbuff_t *tvb, size_t offset, proto_tree *tree) if (computed_cksum == 0) { proto_tree_add_uint_format(mpls_tree, hf_icmp_mpls_checksum, tvb, offset + 2, 2, - cksum, "Checksum: 0x%04x (correct)", cksum); + cksum, "Checksum: 0x%04x [correct]", cksum); } else { @@ -1401,7 +1402,7 @@ dissect_mpls_extensions(tvbuff_t *tvb, size_t offset, proto_tree *tree) proto_tree_add_uint_format(mpls_tree, hf_icmp_mpls_checksum, tvb, offset + 2, 2, cksum, - "Checksum: 0x%04x (incorrect, should be 0x%04x)", + "Checksum: 0x%04x [incorrect, should be 0x%04x]", cksum, in_cksum_shouldbe(cksum, computed_cksum)); } @@ -1769,14 +1770,14 @@ dissect_icmp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (computed_cksum == 0) { proto_tree_add_uint_format(icmp_tree, hf_icmp_checksum, tvb, 2, 2, cksum, - "Checksum: 0x%04x (correct)", cksum); + "Checksum: 0x%04x [correct]", cksum); } else { item = proto_tree_add_boolean(icmp_tree, hf_icmp_checksum_bad, tvb, 2, 2, TRUE); PROTO_ITEM_SET_HIDDEN(item); proto_tree_add_uint_format(icmp_tree, hf_icmp_checksum, tvb, 2, 2, cksum, - "Checksum: 0x%04x (incorrect, should be 0x%04x)", + "Checksum: 0x%04x [incorrect, should be 0x%04x]", cksum, in_cksum_shouldbe(cksum, computed_cksum)); } } else { |