aboutsummaryrefslogtreecommitdiffstats
path: root/wiretap
diff options
context:
space:
mode:
authorGuy Harris <guy@alum.mit.edu>2005-01-14 09:47:22 +0000
committerGuy Harris <guy@alum.mit.edu>2005-01-14 09:47:22 +0000
commitc3240e1ccb2042798f99b4fee774327bb9e188c4 (patch)
tree6b708a89799d3966a1b10bfaec76bdea0f711214 /wiretap
parenta649b53ed32ad7fde77418361be43e5cd128a247 (diff)
Note that the low-order bit of hdr->hdr_2_x.xxx[8] appears to be a "bad
FCS" bit for 802.11, just as it appears to be for Ethernet, and give more details on the 4 bytes of junk at the end of the packet (i.e., that we haven't yet seen an 802.11 capture where it's an FCS rather than just junk). svn path=/trunk/; revision=13028
Diffstat (limited to 'wiretap')
-rw-r--r--wiretap/netxray.c12
1 files changed, 10 insertions, 2 deletions
diff --git a/wiretap/netxray.c b/wiretap/netxray.c
index 678a43327a..bd2051b0c6 100644
--- a/wiretap/netxray.c
+++ b/wiretap/netxray.c
@@ -865,8 +865,16 @@ netxray_set_pseudo_header(wtap *wth, const guint8 *pd, int len,
* It appears, in one 802.11 capture, that
* we have 4 bytes of junk at the ends of
* frames in which hdr->hdr_2_x.xxx[2] and
- * hdr->hdr_2_x.xxx[3] are 0xff; we assume
- * for now that it works like Ethernet.
+ * hdr->hdr_2_x.xxx[3] are 0xff; I haven't
+ * seen any frames where it's an FCS, but,
+ * for now, we still check the fcs_valid
+ * flag - I also haven't seen any capture
+ * where we'd set it based on the realtick
+ * value.
+ *
+ * It also appears that if the low-order bit of
+ * hdr->hdr_2_x.xxx[8] is set, the packet has a
+ * bad FCS.
*/
if (hdr->hdr_2_x.xxx[2] == 0xff &&
hdr->hdr_2_x.xxx[3] == 0xff) {