diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2011-12-23 08:13:42 +0100 |
|---|---|---|
| committer |   John W. Linville <linville@tuxdriver.com> | 2012-01-04 14:30:44 -0500 |
| commit | d71be937202853eda76562e9678073465d5c0fa8 (patch) | |
| tree | dd9161a0a8f35737deedd0ccbdc29ea4651efac4 /drivers/net/wireless | |
| parent | iwlegacy: remove iwl-sta.c (diff) | |
| download | linux-dev-d71be937202853eda76562e9678073465d5c0fa8.tar.xz linux-dev-d71be937202853eda76562e9678073465d5c0fa8.zip | |
iwlegacy: off by one in iwl3945_hw_build_tx_cmd_rate()
We use "rate_index" like this:
rate = iwl3945_rates[rate_index].plcp;
The iwl3945_rates[] array has IWL_RATE_COUNT_3945 elements so the
limit here is off by one.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Stanislaw Gruszka <sgruszka@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless')
| -rw-r--r-- | drivers/net/wireless/iwlegacy/3945.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/iwlegacy/3945.c b/drivers/net/wireless/iwlegacy/3945.c index 863664f9ba8b..0b99deb48426 100644 --- a/drivers/net/wireless/iwlegacy/3945.c +++ b/drivers/net/wireless/iwlegacy/3945.c @@ -683,7 +683,7 @@ il3945_hw_build_tx_cmd_rate(struct il_priv *il, struct il_device_cmd *cmd, struct ieee80211_hdr *hdr, int sta_id, int tx_id) { u16 hw_value = ieee80211_get_tx_rate(il->hw, info)->hw_value; - u16 rate_idx = min(hw_value & 0xffff, RATE_COUNT_3945); + u16 rate_idx = min(hw_value & 0xffff, RATE_COUNT_3945 - 1); u16 rate_mask; int rate; u8 rts_retry_limit; |