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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 18 Mar 2023 20:33:58 -0500 From: Mario Limonciello <mario.limonciello@....com> To: Mika Westerberg <mika.westerberg@...ux.intel.com>, Tom Rix <trix@...hat.com> Cc: andreas.noever@...il.com, michael.jamet@...el.com, YehezkelShB@...il.com, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH] thunderbolt: rename shadowed variables bit to interrupt_bit and auto_clear_bit On 3/16/23 05:37, Mika Westerberg wrote: > On Thu, Mar 16, 2023 at 12:20:48PM +0200, Mika Westerberg wrote: >> +Cc Mario >> >> On Wed, Mar 15, 2023 at 06:04:50PM -0400, Tom Rix wrote: >>> cppcheck reports >>> drivers/thunderbolt/nhi.c:74:7: style: Local variable 'bit' shadows outer variable [shadowVariable] >>> int bit; >>> ^ >>> drivers/thunderbolt/nhi.c:66:6: note: Shadowed declaration >>> int bit = ring_interrupt_index(ring) & 31; >>> ^ >>> drivers/thunderbolt/nhi.c:74:7: note: Shadow variable >>> int bit; >>> ^ >>> For readablity rename the outer to interrupt_bit and the innner >>> to auto_clear_bit. >> Thanks for the patch! Yeah, this did not show up in any of the kbuild >> tests perhaps they are missing cppcheck :( >> >> I'm thinking that I'll just move the two commits from "fixes" to "next" >> and add this one on top (and drop the stable tags) as the code that we >> should be sending to stable should not need additional fixes IMHO. I >> know Mario is on vacation so probably cannot answer here so let's deal >> with this when he is back. > Applied to thunderbolt.git/next (along with the two commits from Mario). Thanks for the fix Tom! Mika - It's unfortunate that a fixup was needed but I'd still like if we can get these 3 commits into 6.3-rc and also to stable.
Powered by blists - more mailing lists