On 01/30/13 13:33, David Brown wrote: > 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++) > I believe that Stephen Boyd is putting some significant work into this > code. Stephen, can you see if this bug is still in your current > version of the code, and make sure that it is fixed there? > This entire file is deleted in my set of patches that moves the current MSM clock code to the common clock framework. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation -- 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
