[PATCH] ARM:mach-msm: seting tail NUL after strncpy
Bjørn Mork
bjorn at mork.no
Wed Jan 30 04:19:16 EST 2013
Chen Gang <gang.chen at asianux.com> writes:
> temp need NUL terminated, or next ptr may cause issue.
>
> Signed-off-by: Chen Gang <gang.chen at asianux.com>
> ---
> arch/arm/mach-msm/clock-debug.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/mach-msm/clock-debug.c
> b/arch/arm/mach-msm/clock-debug.c
> index 4886404..bdca900 100644
> --- a/arch/arm/mach-msm/clock-debug.c
> +++ b/arch/arm/mach-msm/clock-debug.c
> @@ -105,6 +105,8 @@ int __init clock_debug_add(struct clk *clock)
> return -ENOMEM;
>
> strncpy(temp, clock->dbg_name, ARRAY_SIZE(temp)-1);
> + temp[ARRAY_SIZE(temp)-1] = '\0';
> +
> for (ptr = temp; *ptr; ptr++)
> *ptr = tolower(*ptr);
Maybe use strlcpy() instead?
Bjørn
More information about the linux-arm-kernel
mailing list