osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

Call the "no FCS present" version of the Ethernet dissector. 

I haven't found an official spec for EoIB, but slide 10 of

	http://downloads.openfabrics.org/Media/Sonoma2009/Sonoma_2009_Tues_converged-net-bridging.pdf

shows the "Eth Payload" following the "Eth Header" and optional "Vlan
tag", and doesn't show an FCS; "Payload" generally refers to the data
transported by the protocol, which wouldn't include the FCS.

In addition, the capture attached to bug 5061 includes no Ethernet FCS.

So we assume the Ethernet frames carried by EoIB don't include the
Ethernet FCS.

Bug: 9933
Change-Id: I310e5727c42e05498d1f1df08266a48fd6674388
Reviewed-on: https://code.wireshark.org/review/13351
Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
Guy Harris 2016-01-16 20:03:05 -08:00
parent fe82323df0
commit 2c3d343592
1 changed files with 19 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
epan/dissectors/packet-infiniband.c
View File

@ -7456,7 +7456,25 @@ void proto_reg_handoff_infiniband(void)
ipv6_handle = find_dissector("ipv6");
data_handle = find_dissector("data");
eth_handle = find_dissector("eth");
/*
* I haven't found an official spec for EoIB, but slide 10
* of
*
* http://downloads.openfabrics.org/Media/Sonoma2009/Sonoma_2009_Tues_converged-net-bridging.pdf
*
* shows the "Eth Payload" following the "Eth Header" and optional
* "Vlan tag", and doesn't show an FCS; "Payload" generally
* refers to the data transported by the protocol, which wouldn't
* include the FCS.
*
* In addition, the capture attached to bug 5061 includes no
* Ethernet FCS.
*
* So we assume the Ethernet frames carried by EoIB don't include
* the Ethernet FCS.
*/
eth_handle = find_dissector("eth_withoutfcs");
ethertype_dissector_table = find_dissector_table("ethertype");
/* announce an anonymous Infiniband dissector */