Hi Ye,
On Wed, Sep 21, 2022 at 11:45 AM <[email protected]> wrote:
> From: ye xingchen <[email protected]>
>
> The implementation of strscpy() is more robust and safer.
>
> That's now the recommended way to copy NUL terminated strings.
>
> Reported-by: Zeal Robot <[email protected]>
> Signed-off-by: ye xingchen <[email protected]>
Thanks for your patch!
> --- a/arch/um/os-Linux/umid.c
> +++ b/arch/um/os-Linux/umid.c
> @@ -262,7 +262,7 @@ static int __init make_umid(void)
> make_uml_dir();
>
> if (*umid == '\0') {
> - strlcpy(tmp, uml_dir, sizeof(tmp));
> + strscpy(tmp, uml_dir, sizeof(tmp));
> strlcat(tmp, "XXXXXX", sizeof(tmp));
Do we need strscat(), too? ;-)
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
_______________________________________________
linux-um mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-um
