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: Mon, 24 Jun 2019 17:27:43 +0200 From: Peter Zijlstra <peterz@...radead.org> To: Phong Tran <tranmanphong@...il.com> Cc: acme@...nel.org, alexander.shishkin@...ux.intel.com, alexander.sverdlin@...il.com, allison@...utok.net, andrew@...n.ch, ast@...nel.org, bgolaszewski@...libre.com, bpf@...r.kernel.org, daniel@...earbox.net, daniel@...que.org, dmg@...ingmachine.org, festevam@...il.com, gerg@...inux.org, gregkh@...uxfoundation.org, gregory.clement@...tlin.com, haojian.zhuang@...il.com, hsweeten@...ionengravers.com, illusionist.neo@...il.com, info@...ux.net, jason@...edaemon.net, jolsa@...hat.com, kafai@...com, kernel@...gutronix.de, kgene@...nel.org, krzk@...nel.org, kstewart@...uxfoundation.org, linux-arm-kernel@...ts.infradead.org, linux-imx@....com, linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org, linux-samsung-soc@...r.kernel.org, linux@...linux.org.uk, liviu.dudau@....com, lkundrak@...sk, lorenzo.pieralisi@....com, mark.rutland@....com, mingo@...hat.com, namhyung@...nel.org, netdev@...r.kernel.org, nsekhar@...com, robert.jarzmik@...e.fr, s.hauer@...gutronix.de, sebastian.hesselbarth@...il.com, shawnguo@...nel.org, songliubraving@...com, sudeep.holla@....com, swinslow@...il.com, tglx@...utronix.de, tony@...mide.com, will@...nel.org, yhs@...com Subject: Re: [PATCH V2 00/15] cleanup cppcheck signed shifting errors On Mon, Jun 24, 2019 at 08:50:50PM +0700, Phong Tran wrote: > There are errors with cppcheck > > "Shifting signed 32-bit value by 31 bits is undefined behaviour errors" As I've already told you; your checker is bad. That is not in face undefined behaviour in the kernel. That's not to say you shouldn't clean up the code, but don't give broken checkout output as a reason.
Powered by blists - more mailing lists