Thread (53 messages) flat view 53 messages, 9 authors, 4d ago

Re: [PATCH 25/28] EDAC/thunderx: Replace strlcat() with seq_buf

From: Borislav Petkov <bp@alien8.de>
Date: 2026-09-16 01:11:51
Also in: dri-devel, intel-wired-lan, intel-xe, linux-devicetree, linux-edac, linux-gpio, linux-hardening, linux-media, linux-mips, linux-nfs, linux-samsung-soc, linux-scsi, linux-sound, linux-wireless, lkml, loongarch, netdev

On Tue, Sep 15, 2026 at 01:00:00PM -0700, Kees Cook wrote:
Seems like seq_buf would be better her? But yes, at the very least, this
needs to be scnprintf, not snprintf (or add the checks as you have
here).

The rest of the seq_buf conversions look right.
This driver has been an orphan for a while now (we just made it official) so
I'd prefer it if we don't do anything here.

And yes, that mega CC list is just gross.

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help