diff options
| author | Ben Whitten <ben.whitten@gmail.com> | 2015-12-30 13:05:58 +0000 |
|---|---|---|
| committer | Tom Rini <trini@konsulko.com> | 2016-01-14 22:11:34 -0500 |
| commit | 192bc6948b02ff4168cab16162fffb507946dc2b (patch) | |
| tree | b49cf85a3fa910182ce7dc2508f00ccb8ade03d4 /board/keymile | |
| parent | 4edde96111aefac63d6aaca6ba87a90d149e973e (diff) | |
| download | talos-obmc-uboot-192bc6948b02ff4168cab16162fffb507946dc2b.tar.gz talos-obmc-uboot-192bc6948b02ff4168cab16162fffb507946dc2b.zip | |
Fix GCC format-security errors and convert sprintfs.
With format-security errors turned on, GCC picks up the use of sprintf with
a format parameter not being a string literal.
Simple uses of sprintf are also converted to use strcpy.
Signed-off-by: Ben Whitten <ben.whitten@gmail.com>
Acked-by: Wolfgang Denk <wd@denx.de>
Reviewed-by: Tom Rini <trini@konsulko.com>
Diffstat (limited to 'board/keymile')
| -rw-r--r-- | board/keymile/common/common.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/board/keymile/common/common.c b/board/keymile/common/common.c index 8de129dc83..a42f3eca33 100644 --- a/board/keymile/common/common.c +++ b/board/keymile/common/common.c @@ -168,7 +168,7 @@ static int do_setboardid(cmd_tbl_t *cmdtp, int flag, int argc, printf("can't get the IVM_Boardid\n"); return 1; } - sprintf((char *)buf, "%s", p); + strcpy((char *)buf, p); setenv("boardid", (char *)buf); printf("set boardid=%s\n", buf); @@ -177,7 +177,7 @@ static int do_setboardid(cmd_tbl_t *cmdtp, int flag, int argc, printf("can't get the IVM_HWKey\n"); return 1; } - sprintf((char *)buf, "%s", p); + strcpy((char *)buf, p); setenv("hwkey", (char *)buf); printf("set hwkey=%s\n", buf); printf("Execute manually saveenv for persistent storage.\n"); |
