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);
--
With Best Regards,
Andy Shevchenko
