ARM: vf610: use strcpy for soc environment variable
authorStefan Agner <stefan.agner@toradex.com>
Wed, 28 Sep 2016 09:29:28 +0000 (11:29 +0200)
committerStefano Babic <sbabic@denx.de>
Fri, 7 Oct 2016 13:56:52 +0000 (15:56 +0200)
To create the soc environment variable we concatenate two strings
on the stack. So far, strcat has been used for the first string as
well as for the second string. Since the variable on the stack is
not initialized, the first strcat may not start using the first
entry in the character array. This then could lead to an buffer
overflow on the stack.

Signed-off-by: Stefan Agner <stefan.agner@toradex.com>
Acked-by: Marcel Ziswiler <marcel.ziswiler@toradex.com>
arch/arm/cpu/armv7/vf610/generic.c

index 08b9ef42a19012982a44f9572198931da3681aa6..50eb0c6b97c0262dbe1d3a34430f4fb78fd7ec79 100644 (file)
@@ -322,7 +322,7 @@ int arch_misc_init(void)
 {
        char soc[6];
 
-       strcat(soc, "vf");
+       strcpy(soc, "vf");
        strcat(soc, soc_type);
        setenv("soc", soc);