diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2012-04-18 07:22:12 +0000 |
|---|---|---|
| committer | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2012-04-18 07:22:12 +0000 |
| commit | 6aedbbaf3503690c45962c0058dd348a506b3e5a (patch) | |
| tree | b4afcda5dfbe979183fa4ce7819eafd69b11c0d4 /wiretap/wtap.h | |
| parent | e6b7af69b566ca4cbb52922f79ce2a8d32dea02e (diff) | |
From Colus Tang via https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=7134
Unable to display the correct IEEE802.11 MCS data rates due to header definition
The problem is due to the ieee_802_11_phdr.data_rate is defined as guint8,
since this variable is counting number of 0.5Mbps units, any datarates which is
higher than 255Mbps would get wrapped up. In the above example, only the lower
8bit value will be put into the ieee_802_11_phdr which is 0x04 and result in
the incorrect 2Mbps display.
There are 802.11n WLAN product is capable to transmit @450Mbps, we should fix
this data_rate from guint8 to guint16.
#BACKPORT
svn path=/trunk/; revision=42123
Diffstat (limited to 'wiretap/wtap.h')
| -rw-r--r-- | wiretap/wtap.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/wiretap/wtap.h b/wiretap/wtap.h index 505ab31cb7..c049cce071 100644 --- a/wiretap/wtap.h +++ b/wiretap/wtap.h @@ -514,7 +514,7 @@ struct p2p_phdr { struct ieee_802_11_phdr { gint fcs_len; /* Number of bytes of FCS - -1 means "unknown" */ guint8 channel; /* Channel number */ - guint8 data_rate; /* in .5 Mb/s units */ + guint16 data_rate; /* in .5 Mb/s units */ guint8 signal_level; /* percentage */ }; |