On 22/05/2022 17:11, 陈学兵 wrote:
>
> The strlcpy should not be used because it doesn't limit the source
> length. Preferred is strscpy.
>
> Signed-off-by: XueBing Chen <chenxuebing@xxxxxxx>
Your name in From field does not match exactly one used in
Signed-off-by. Name might be actually the same, but different alphabet
is used. Please use the same one - latin transliteration - in both cases.
> ---
> arch/arm/mach-s3c/mach-mini2440.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-s3c/mach-mini2440.c b/arch/arm/mach-s3c/mach-mini2440.c
> index 551ec660ab59..98cebc06521c 100644
> --- a/arch/arm/mach-s3c/mach-mini2440.c
> +++ b/arch/arm/mach-s3c/mach-mini2440.c
> @@ -624,7 +624,7 @@ static char mini2440_features_str[12] __initdata = "0tb";
> static int __init mini2440_features_setup(char *str)
> {
> if (str)
> - strlcpy(mini2440_features_str, str,
> + strscpy(mini2440_features_str, str,
> sizeof(mini2440_features_str));
> return 1;
> }
Best regards,
Krzysztof
