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
| ||
|
Message-ID: <CAFhGd8qAfWiC0en-VXaR_DxNr+xFfw8zwUJ4KgCd8ieSmU3t5g@mail.gmail.com> Date: Thu, 12 Oct 2023 15:01:06 -0700 From: Justin Stitt <justinstitt@...gle.com> To: Andrew Lunn <andrew@...n.ch> Cc: Heiner Kallweit <hkallweit1@...il.com>, Russell King <linux@...linux.org.uk>, "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, linux-hardening@...r.kernel.org Subject: Re: [PATCH] net: mdio: replace deprecated strncpy with strscpy On Thu, Oct 12, 2023 at 2:59 PM Andrew Lunn <andrew@...n.ch> wrote: > > On Thu, Oct 12, 2023 at 09:53:03PM +0000, Justin Stitt wrote: > > strncpy() is deprecated for use on NUL-terminated destination strings > > [1] and as such we should prefer more robust and less ambiguous string > > interfaces. > > Hi Justin > > You just sent two patches with the same Subject. That got me confused > for a while, is it a resend? A new version? Yep, just saw this. I'm working (top to bottom) on a list of strncpy hits. I have an automated tool fetch the prefix and update the subject line accordingly. They are two separate patches but ended up with the same exact subject line due to oversight and over-automation. Looking for guidance: Should I combine them into one patch? > > > Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings [1] > > Link: https://manpages.debian.org/testing/linux-manual-4.8/strscpy.9.en.html [2] > > Link: https://github.com/KSPP/linux/issues/90 > > Cc: linux-hardening@...r.kernel.org > > Signed-off-by: Justin Stitt <justinstitt@...gle.com> > > Reviewed-by: Andrew Lunn <andrew@...n.ch> > > Andrew > Thanks Justin
Powered by blists - more mailing lists