diff options
| author | Peng Fan <peng.fan@nxp.com> | 2016-01-10 12:52:15 +0800 |
|---|---|---|
| committer | guoyin.chen <guoyin.chen@freescale.com> | 2016-03-04 15:53:39 +0800 |
| commit | 72f69ee87cb88d4e18fe6b05701bff275dc1b68f (patch) | |
| tree | bbdb684ebc5caf738d0aa1e9934c758e58d1a01d | |
| parent | 0b2918d4a14e8a9ed83406d89624ae2442ec20b7 (diff) | |
| download | u-boot-imx-72f69ee87cb88d4e18fe6b05701bff275dc1b68f.zip u-boot-imx-72f69ee87cb88d4e18fe6b05701bff275dc1b68f.tar.gz u-boot-imx-72f69ee87cb88d4e18fe6b05701bff275dc1b68f.tar.bz2 | |
MLK-12200 common: cli_simple: use strlcpy instead of strcpy
Report Coverity log:
Destination buffer too small (STRING_OVERFLOW)
string_overflow: You might overrun the 1024 byte destination string
lastcommand by writing 1025 bytes from console_buffer
Signed-off-by: Peng Fan <peng.fan@nxp.com>
| -rw-r--r-- | common/cli_simple.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/common/cli_simple.c b/common/cli_simple.c index 6c65cc6..690f00b 100644 --- a/common/cli_simple.c +++ b/common/cli_simple.c @@ -275,7 +275,8 @@ void cli_simple_loop(void) flag = 0; /* assume no special flags for now */ if (len > 0) - strcpy(lastcommand, console_buffer); + strlcpy(lastcommand, console_buffer, + CONFIG_SYS_CBSIZE + 1); else if (len == 0) flag |= CMD_FLAG_REPEAT; #ifdef CONFIG_BOOT_RETRY_TIME |