Re: git: b1a9e570fe2d - main - linuxkpi: Add `strscpy_pad()` to <linux/string.h>

On 15 Feb 2023, at 22:26, Jean-Sébastien Pédron <dumbbell@FreeBSD.org> wrote:
> 
> The branch main has been updated by dumbbell (ports committer):
> 
> URL: https://cgit.FreeBSD.org/src/commit/?id=b1a9e570fe2da0a43c41327d7ce21b27bf8257b8
> 
> commit b1a9e570fe2da0a43c41327d7ce21b27bf8257b8
> Author:     Jean-Sébastien Pédron <dumbbell@FreeBSD.org>
> AuthorDate: 2023-02-14 23:46:13 +0000
> Commit:     Jean-Sébastien Pédron <dumbbell@FreeBSD.org>
> CommitDate: 2023-02-15 22:19:41 +0000
> 
>    linuxkpi: Add `strscpy_pad()` to <linux/string.h>
> 
>    It's the same as `strscpy()` except that it fills the rest of the
>    destination buffer with zeroes if the source buffer is shorter.
> 
>    Reviewed by:    manu
>    Approved by:    manu
> ---
> sys/compat/linuxkpi/common/include/linux/string.h | 9 +++++++++
> 1 file changed, 9 insertions(+)
> 
> diff --git a/sys/compat/linuxkpi/common/include/linux/string.h b/sys/compat/linuxkpi/common/include/linux/string.h
> index 65422cac629c..20c3729aaf12 100644
> --- a/sys/compat/linuxkpi/common/include/linux/string.h
> +++ b/sys/compat/linuxkpi/common/include/linux/string.h
> @@ -207,6 +207,15 @@ strscpy(char* dst, const char* src, size_t len)
> 	return (-E2BIG);
> }
> 
> +static inline ssize_t
> +strscpy_pad(char* dst, const char* src, size_t len)
> +{
> +

style(9) doesn’t need this blank line any more, please stop including
them in your commits.

Jess

> +	bzero(dst, len);
> +
> +	return (strscpy(dst, src, len));
> +}
> +
> static inline void *
> memset32(uint32_t *b, uint32_t c, size_t len)
> {