Chen Gang <gang.chen@xxxxxxxxxxx> writes: > temp need NUL terminated, or next ptr may cause issue. > > Signed-off-by: Chen Gang <gang.chen@xxxxxxxxxxx> > --- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html