Re: [PATCH] s390/sclp: replace deprecated strncpy with strtomem

From: Vasily Gorbik
Date: Wed Oct 25 2023 - 06:23:35 EST


On Mon, Oct 23, 2023 at 07:14:49PM +0000, Justin Stitt wrote:
> Let's move away from using strncpy() as it is deprecated [1].
>
> Instead use strtomem() as `e.id` is already marked as nonstring:
> | char id[4] __nonstring;
>
> We don't need strtomem_pad() because `e` is already memset to 0 --
> rendering any additional NUL-padding useless.
>
> Link: https://www.kernel.org/doc/html/latest/process/deprecated.html#strncpy-on-nul-terminated-strings [1]
> Link: https://github.com/KSPP/linux/issues/90
> Cc: linux-hardening@xxxxxxxxxxxxxxx
> Signed-off-by: Justin Stitt <justinstitt@xxxxxxxxxx>
> ---
> Note: build-tested only.
>
> Found with: $ rg "strncpy\("
> ---
> drivers/s390/char/sclp.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)

Applied, thank you!