diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-06-06 03:43:00 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2013-06-10 14:41:26 -0400 |
commit | f28bc92c9530fe39eccfcff392879f19bd234578 (patch) | |
tree | e921a736f67585ceda56149f87dd7ebdb9857056 /drivers/net/wireless/cw1200 | |
parent | aa63e18e3ddad4eb15d4af34ae66e7f4dcc7a6d0 (diff) |
cw1200: read beyond end of array in debug code
This has only one caller and rates[] is an array with
IEEE80211_TX_MAX_RATES (4) elements.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/cw1200')
-rw-r--r-- | drivers/net/wireless/cw1200/txrx.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/net/wireless/cw1200/txrx.c b/drivers/net/wireless/cw1200/txrx.c index 0e40890e1993..a2547473ec5c 100644 --- a/drivers/net/wireless/cw1200/txrx.c +++ b/drivers/net/wireless/cw1200/txrx.c | |||
@@ -190,13 +190,12 @@ static void tx_policy_build(const struct cw1200_common *priv, | |||
190 | policy->retry_count += retries; | 190 | policy->retry_count += retries; |
191 | } | 191 | } |
192 | 192 | ||
193 | pr_debug("[TX policy] Policy (%zu): %d:%d, %d:%d, %d:%d, %d:%d, %d:%d\n", | 193 | pr_debug("[TX policy] Policy (%zu): %d:%d, %d:%d, %d:%d, %d:%d\n", |
194 | count, | 194 | count, |
195 | rates[0].idx, rates[0].count, | 195 | rates[0].idx, rates[0].count, |
196 | rates[1].idx, rates[1].count, | 196 | rates[1].idx, rates[1].count, |
197 | rates[2].idx, rates[2].count, | 197 | rates[2].idx, rates[2].count, |
198 | rates[3].idx, rates[3].count, | 198 | rates[3].idx, rates[3].count); |
199 | rates[4].idx, rates[4].count); | ||
200 | } | 199 | } |
201 | 200 | ||
202 | static inline bool tx_policy_is_equal(const struct tx_policy *wanted, | 201 | static inline bool tx_policy_is_equal(const struct tx_policy *wanted, |