Subject: [PATCH V2] ath6kl: Fix bug in computing AMSU subframe padding

This fixes AMSDU rx, otherwise it fails with the following warnings.

"802.3 AMSDU frame bound check failed"

Signed-off-by: Vasanthakumar Thiagarajan <[email protected]>
---
drivers/net/wireless/ath/ath6kl/txrx.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)

V2 -- It is 4-byte aligned not 3, thanks to Kevin for pointing this out.

diff --git a/drivers/net/wireless/ath/ath6kl/txrx.c b/drivers/net/wireless/ath/ath6kl/txrx.c
index 5d3d4b6..44bf227 100644
--- a/drivers/net/wireless/ath/ath6kl/txrx.c
+++ b/drivers/net/wireless/ath/ath6kl/txrx.c
@@ -812,7 +812,7 @@ static void aggr_slice_amsdu(struct aggr_info *p_aggr,
/* Add the length of A-MSDU subframe padding bytes -
* Round to nearest word.
*/
- frame_8023_len = ALIGN(frame_8023_len + 3, 3);
+ frame_8023_len = ALIGN(frame_8023_len, 4);

framep += frame_8023_len;
amsdu_len -= frame_8023_len;
--
1.7.0.4



2011-08-17 16:32:36

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH V2] ath6kl: Fix bug in computing AMSU subframe padding

On 08/15/2011 10:49 PM, Vasanthakumar Thiagarajan wrote:
> This fixes AMSDU rx, otherwise it fails with the following warnings.
>
> "802.3 AMSDU frame bound check failed"

Thanks, applied.

Kalle