[PATCH v2] drm/etnaviv: refactor deprecated strncpy

Kees Cook keescook at chromium.org
Sun Sep 24 02:53:41 UTC 2023


On Mon, Sep 18, 2023 at 01:34:08PM +0000, Justin Stitt wrote:
> `strncpy` is deprecated for use on NUL-terminated destination strings [1].
> 
> We should prefer more robust and less ambiguous string interfaces.
> 
> A suitable replacement is `strscpy_pad` due to the fact that it
> guarantees NUL-termination on the destination buffer whilst maintaining
> the NUL-padding behavior that strncpy provides.
> 
> 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 at vger.kernel.org
> Cc: Bo YU <tsu.yubo at gmail.com>
> Signed-off-by: Justin Stitt <justinstitt at google.com>

Looks good to me now. Thanks!

Reviewed-by: Kees Cook <keescook at chromium.org>

(Though again if you need a v3, making the Subject more specific would
be nice, "...: Replace strncpy with strscpy_pad"

-Kees

-- 
Kees Cook


More information about the etnaviv mailing list