Search Linux Wireless

Re: [PATCH] lib80211: Introduce TKIP_HDR_LEN define for code clarity

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Feb 02, 2010 at 01:11:10PM +0200, Andriy Tkachuk wrote:
> Introduce TKIP_HDR_LEN define for code clarity (in the same way as
> CCMP_HDR_LEN).
> 
> Also odd len variable (not used) dropped from lib80211_tkip_hdr().
> 
> Signed-off-by: Andriy V. Tkachuk <andrit@xxxxxxx>
> 
> ---
>  net/wireless/lib80211_crypt_tkip.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/net/wireless/lib80211_crypt_tkip.c
> b/net/wireless/lib80211_crypt_tkip.c
> index c362873..c4fb4fb 100644
> --- a/net/wireless/lib80211_crypt_tkip.c
> +++ b/net/wireless/lib80211_crypt_tkip.c
> @@ -36,6 +36,8 @@ MODULE_AUTHOR("Jouni Malinen");
>  MODULE_DESCRIPTION("lib80211 crypt: TKIP");
>  MODULE_LICENSE("GPL");
> 
> +#define TKIP_HDR_LEN 8
> +
>  struct lib80211_tkip_data {
>  #define TKIP_KEY_LEN 32
>  	u8 key[TKIP_KEY_LEN];
> @@ -314,13 +316,12 @@ static int lib80211_tkip_hdr(struct sk_buff *skb,
> int hdr_len,

After a quick look, this and the other patch look fine.  But, the
patches are whitespace-damaged.  I can fix these, but in the future
please be sure to use a mailer that does not break lines.  I recommend
mutt and/or 'git send-email'.

Thanks,

John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@xxxxxxxxxxxxx			might be all we have.  Be ready.
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux