linux-kernel - Re: [PATCH 2/2] Staging: RT2860: Fixed multiple errors in rtusb

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 9 Nov 2010 13:23:22 -0800
From:	Greg KH <greg@...ah.com>
To:	Neil Munro <neilmunro@...il.com>
Cc:	gregkh@...e.de, wfp5p@...ginia.edu, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH 2/2] Staging: RT2860: Fixed multiple errors in
 rtusb_io.h

On Sat, Oct 23, 2010 at 08:12:17PM +0100, Neil Munro wrote:
> Fixed the errors in rtusb_io.h

"errors"?  What type of errors?  Do you mean checkpatch.pl
warnings/errors?

> 
> Signed-off-by: Neil Munro <neilmunro@...il.com>
> ---
>  drivers/staging/rt2860/rtusb_io.h |   44 +++++++++++++++++++-----------------
>  1 files changed, 23 insertions(+), 21 deletions(-)
> 
> diff --git a/drivers/staging/rt2860/rtusb_io.h b/drivers/staging/rt2860/rtusb_io.h
> index 64a2fe4..bbb34d2 100644
> --- a/drivers/staging/rt2860/rtusb_io.h
> +++ b/drivers/staging/rt2860/rtusb_io.h
> @@ -144,42 +144,44 @@ struct rt_cmdq {
> 
>  /* add Client security information into ASIC WCID table and IVEIV table */
>  #define RTMP_STA_SECURITY_INFO_ADD(pAd, apidx, KeyID, pEntry)						\
> -	{	RTMP_STA_ENTRY_MAC_RESET(pAd, pEntry->Aid);								\
> -		if (pEntry->Aid >= 1) {														\
> -			struct rt_set_asic_wcid_attri	SetAsicWcidAttri;								\
> -			SetAsicWcidAttri.WCID = pEntry->Aid;									\
> -			if ((pEntry->AuthMode <= Ndis802_11AuthModeAutoSwitch) &&				\
> -				(pEntry->WepStatus == Ndis802_11Encryption1Enabled))				\
> -			{																		\
> -				SetAsicWcidAttri.Cipher = pAd->SharedKey[apidx][KeyID].CipherAlg;	\
> -			}																		\
> -			else if (pEntry->AuthMode == Ndis802_11AuthModeWPANone)					\
> -			{																		\
> -				SetAsicWcidAttri.Cipher = pAd->SharedKey[apidx][KeyID].CipherAlg;	\
> -			}																		\
> -			else SetAsicWcidAttri.Cipher = 0;										\
> -            DBGPRINT(RT_DEBUG_TRACE, ("aid cipher = %ld\n",SetAsicWcidAttri.Cipher));       \
> +{
> +	RTMP_STA_ENTRY_MAC_RESET(pAd, pEntry->Aid);								\
> +	if (pEntry->Aid >= 1) {														\
> +		struct rt_set_asic_wcid_attri	SetAsicWcidAttri;								\
> +		SetAsicWcidAttri.WCID = pEntry->Aid;									\
> +		if ((pEntry->AuthMode <= Ndis802_11AuthModeAutoSwitch) &&				\

Are you sure this is still all ok?  The tabs look pretty wrong and
the ends of the line should be smaller, right?

If you are going to fix up the spacing, please also fix the length of
the lines at the same time, especially when it is so easy to do so.

Care to resend this?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/