linux-kernel - Re: [PATCH] staging: rtl8712: remove Unnecessary parentheses

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:   Wed, 06 Apr 2022 11:08:54 -0700
From:   Joe Perches <joe@...ches.com>
To:     Julia Lawall <julia.lawall@...ia.fr>,
        Alaa Mohamed <eng.alaamohamedsoliman.am@...il.com>
Cc:     outreachy@...ts.linux.dev, Larry.Finger@...inger.net,
        florian.c.schilhabel@...glemail.com, gregkh@...uxfoundation.org,
        linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] staging: rtl8712: remove Unnecessary parentheses

On Wed, 2022-04-06 at 17:37 +0200, Julia Lawall wrote:
> If you want to capitalize something in the subject, it could be the first
> word.  Capitalizing the second word is a bit strange.
> 
> On Wed, 6 Apr 2022, Alaa Mohamed wrote:
> 
> > Reported by checkpatch:
> > 
> > CHECK: Unnecessary parentheses
> 
> Indicating that the problem was detected by checkpatch is good.  But
> actually, the parentheses all have the same property.  So you could use
> the log message to describe what kind of unnecessary parentheses were
> removed.  That would help the maintainer know what to look for.
[]

> > diff --git a/drivers/staging/rtl8712/rtl871x_ioctl_linux.c b/drivers/staging/rtl8712/rtl871x_ioctl_linux.c
[]
> > @@ -82,9 +82,9 @@ static inline void handle_pairwise_key(struct sta_info *psta,
> >  	       (param->u.crypt. key_len > 16 ? 16 : param->u.crypt.key_len));
> >  	if (strcmp(param->u.crypt.alg, "TKIP") == 0) { /* set mic key */
> >  		memcpy(psta->tkiptxmickey. skey,

Also, the space after the periods here are not great either.

		memcpy(psta->tkiptxmickey. skey,
vs
		memcpy(psta->tkiptxmickey.skey,