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
| ||
|
Date: Thu, 11 May 2017 16:31:13 +0200 From: Arnd Bergmann <arnd@...db.de> To: "Mintz, Yuval" <Yuval.Mintz@...ium.com> Cc: "David S. Miller" <davem@...emloft.net>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [PATCH] qed: fix uninitialized data in aRFS intrastructure On Thu, May 11, 2017 at 4:03 PM, Mintz, Yuval <Yuval.Mintz@...ium.com> wrote: >> register, which went subtly wrong due to the wrong size in a memset(): >> >> ethernet/qlogic/qed/qed_init_fw_funcs.c: In function >> 'qed_set_rfs_mode_disable': >> ethernet/qlogic/qed/qed_init_fw_funcs.c:993:3: error: '*((void >> *)&ramline+4)' is used uninitialized in this function [-Werror=uninitialized] >> >> This removes the silly loop and memset, and instead directly writes the >> correct value to the register. > > Hi Arnd, > > For the most part - I'm almost all in favor of this change. > But just to make it clear - the actual fix could have been a one-liner, right? > The rest are style changes. Correct. Having the correct length in the memset is a sufficient fix for the warning, but it felt wrong to send it since the root of the problem seems to be the complexity of the code that was hiding it. >> +#define CAM_REG(pf_id) (PRS_REG_GFT_CAM + CAM_LINE_SIZE * (pf_id)) >> +#define RAM_REG(pf_id) (PRS_REG_GFT_PROFILE_MASK_RAM + > > Not sure I'm a huge fan of this specific style change; > Seems like we could easily manage without these macros. I tried first and ended up with really long lines that I did not like. Generally speaking, feel free to treat any of my compile-time warning fix patches as simple bug reports and apply a different fix that seems more appropriate. I mainly send it in patch form since that seems to be the quickest way to address any issues. Arnd
Powered by blists - more mailing lists