diff options
author | Bill Meier <wmeier@newsguy.com> | 2013-01-29 13:59:17 +0000 |
---|---|---|
committer | Bill Meier <wmeier@newsguy.com> | 2013-01-29 13:59:17 +0000 |
commit | ef242060b8c38eb0f9c106246e49f9fadbe5767c (patch) | |
tree | b82ea9cd2a7cec2d10b91329c1346e74be9977ff /epan/dissectors/packet-ixveriwave.c | |
parent | 3520ab3889025aaaaf7228dee9abfe1669aa60bb (diff) |
Add an XXX comment about some code which will never be executed;
Also: mention that there's a missing hf[] entry.
svn path=/trunk/; revision=47342
Diffstat (limited to 'epan/dissectors/packet-ixveriwave.c')
-rw-r--r-- | epan/dissectors/packet-ixveriwave.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/epan/dissectors/packet-ixveriwave.c b/epan/dissectors/packet-ixveriwave.c index c2358928ef..c5b85bc627 100644 --- a/epan/dissectors/packet-ixveriwave.c +++ b/epan/dissectors/packet-ixveriwave.c @@ -947,6 +947,9 @@ wlantap_dissect(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, proto_tree /* If we had an in-header FCS, check it. */ if (hdr_fcs_ti) { + /* XXX: 'hdr_fcs_ti' is never set so the following code will never be executed ? */ + /* Also: 'sent fcs' always 0 ? */ + /* 'hf_radiotap_fcs_bad' has no entry in hf[] */ /* It would be very strange for the header to have an FCS for the * frame *and* the frame to have the FCS at the end, but it's possible, so * take that into account by using the FCS length recorded in pinfo. */ |