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:35:54 +0800 |
| commit | eb8b3f27e2937d8856fe01441b6c2880b59f02b5 (patch) | |
| tree | bbdb684ebc5caf738d0aa1e9934c758e58d1a01d | |
| parent | 18f44905a030a4c16e5740da3e74160e5966e101 (diff) | |
| download | u-boot-imx-eb8b3f27e2937d8856fe01441b6c2880b59f02b5.zip u-boot-imx-eb8b3f27e2937d8856fe01441b6c2880b59f02b5.tar.gz u-boot-imx-eb8b3f27e2937d8856fe01441b6c2880b59f02b5.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 |