diff options
| author |   Karl Beldan <karl.beldan@rivierawaves.com> | 2014-10-22 18:20:37 +0200 |
|---|---|---|
| committer |   Johannes Berg <johannes.berg@intel.com> | 2014-10-23 20:36:13 +0200 |
| commit | 8ec7886b1cd59c6e76a6d8fa413f9d338cfedc96 (patch) | |
| tree | 8d0e02477ec4079f0ce373dd3e44e31366fe1d89 /net/mac80211/wep.c | |
| parent | mac80211: don't remove tainted keys after not programming (diff) | |
| download | linux-dev-8ec7886b1cd59c6e76a6d8fa413f9d338cfedc96.tar.xz linux-dev-8ec7886b1cd59c6e76a6d8fa413f9d338cfedc96.zip | |
mac80211: minstrel_ht: use group flags instead of index to display rates
When displaying a rate through debugfs minstrel_ht guesses its flags
comparing group indexes. Since 3ec373c421b6 ("mac80211: minstrel_ht:
include type (cck/ht) in rates flag"), the rate flags of interest are
present in the mcs_group-s, so use it.
While improving the code, this also fixes a smatch false positive
"error: testing array offset 'i' after use" in minstrel_ht_stats_dump.
This warning only triggers after 9208247d74bc ("mac80211: minstrel_ht:
add basic support for VHT rates <= 3SS@80MHz") with
CONFIG_MAC80211_RC_MINSTREL_VHT unset because then MINSTREL_VHT_GROUP_0
is above MINSTREL_GROUPS_NB and smatch only barks when the "testing
array offset" seems to prevent possible out of bonds accesses (which
does not happen here since i < ARRAY_SIZE(mi->groups)).
Signed-off-by: Karl Beldan <karl.beldan@rivierawaves.com>
Cc: Felix Fietkau <nbd@openwrt.org>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/mac80211/wep.c')
0 files changed, 0 insertions, 0 deletions