On Thu, Sep 17, 2026 at 03:50:23PM +0300, Andy Shevchenko wrote: > On Wed, Sep 16, 2026 at 05:23:18PM -0700, Kees Cook wrote: > > perf_stats_show() and flags_show() build their output with a seq_buf > > and return seq_buf_used(), which may include the trailing NUL byte > > when the seq_buf has overflowed. Use seq_buf_strlen() instead. > > > > Build tested ARCH=powerpc ppc64_defconfig with GCC powerpc64-linux-gnu > > 16.1.0: > > arch/powerpc/platforms/pseries/papr_scm.o > > ... > > > free_stats: > > kfree(stats); > > - return rc ? rc : (ssize_t)seq_buf_used(&s); > > + return rc ? rc : (ssize_t)seq_buf_strlen(&s); > > } > > On a brief look it's a single ternary like this in the file, perhaps while at > it move to Elvis op? > > return rc ?: (ssize_t)seq_buf_strlen(&s);
Sure! I've updated this for v2. -- Kees Cook
