forked from osmocom/wireshark
A follow-up to r50127 to highlight the correct 4 bytes of the Ethernet FCS for the hf_eth_fcs_good and hf_eth_fcs_bad fields.
svn path=/trunk/; revision=50128
This commit is contained in:
parent
00b1a9df1a
commit
93002cb9c5
|
@ -621,11 +621,11 @@ add_ethernet_trailer(packet_info *pinfo, proto_tree *tree, proto_tree *fh_tree,
|
|||
padding_length+trailer_length, 4, sent_fcs,
|
||||
"Frame check sequence: 0x%08x [correct]", sent_fcs);
|
||||
checksum_tree = proto_item_add_subtree(item, ett_eth_fcs);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, tvb,
|
||||
trailer_length, 2, TRUE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, trailer_tvb,
|
||||
padding_length+trailer_length, 4, TRUE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, tvb,
|
||||
trailer_length, 2, FALSE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, trailer_tvb,
|
||||
padding_length+trailer_length, 4, FALSE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
} else {
|
||||
item = proto_tree_add_uint_format(fh_tree, hf_eth_fcs, trailer_tvb,
|
||||
|
@ -633,11 +633,11 @@ add_ethernet_trailer(packet_info *pinfo, proto_tree *tree, proto_tree *fh_tree,
|
|||
"Frame check sequence: 0x%08x [incorrect, should be 0x%08x]",
|
||||
sent_fcs, fcs);
|
||||
checksum_tree = proto_item_add_subtree(item, ett_eth_fcs);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, tvb,
|
||||
trailer_length, 2, FALSE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, trailer_tvb,
|
||||
padding_length+trailer_length, 4, FALSE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, tvb,
|
||||
trailer_length, 2, TRUE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, trailer_tvb,
|
||||
padding_length+trailer_length, 4, TRUE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
expert_add_info(pinfo, item, &ei_eth_fcs_bad);
|
||||
col_append_str(pinfo->cinfo, COL_INFO, " [ETHERNET FRAME CHECK SEQUENCE INCORRECT]");
|
||||
|
@ -647,11 +647,11 @@ add_ethernet_trailer(packet_info *pinfo, proto_tree *tree, proto_tree *fh_tree,
|
|||
padding_length+trailer_length, 4, sent_fcs,
|
||||
"Frame check sequence: 0x%08x [validiation disabled]", sent_fcs);
|
||||
checksum_tree = proto_item_add_subtree(item, ett_eth_fcs);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, tvb,
|
||||
trailer_length, 2, FALSE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_good, trailer_tvb,
|
||||
padding_length+trailer_length, 4, FALSE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, tvb,
|
||||
trailer_length, 2, FALSE);
|
||||
item = proto_tree_add_boolean(checksum_tree, hf_eth_fcs_bad, trailer_tvb,
|
||||
padding_length+trailer_length, 4, FALSE);
|
||||
PROTO_ITEM_SET_GENERATED(item);
|
||||
}
|
||||
trailer_length += 4;
|
||||
|
|
Loading…
Reference in New Issue