[U-Boot] [PATCH v1 1/1] fix: possible overflow when copying the string

[U-Boot] [PATCH v1 1/1] fix: possible overflow when copying the string

[Imran Zaman]

Bigger source buffer than dest buffer could
overflow when copying strings.
Source and destination buffer sizes are same now.

Signed-off-by: Imran Zaman <imran.zaman@intel.com>
---
 common/cli_simple.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/common/cli_simple.c b/common/cli_simple.c
index 00a8d2f..d8b40c9 100644
--- a/common/cli_simple.c
+++ b/common/cli_simple.c
@@ -258,7 +258,7 @@ int cli_simple_run_command(const char *cmd, int flag)
 
 void cli_simple_loop(void)
 {
-	static char lastcommand[CONFIG_SYS_CBSIZE] = { 0, };
+	static char lastcommand[CONFIG_SYS_CBSIZE + 1] = { 0, };
 
 	int len;
 	int flag;
-- 
1.9.1

---------------------------------------------------------------------
Intel Finland Oy
Registered Address: PL 281, 00181 Helsinki 
Business Identity Code: 0357606 - 4 
Domiciled in Helsinki 

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

[U-Boot] [U-Boot, v1, 1/1] fix: possible overflow when copying the string

[Tom Rini]

On Mon, Sep 07, 2015 at 11:24:08AM +0300, Imran Zaman wrote:

> Bigger source buffer than dest buffer could
> overflow when copying strings.
> Source and destination buffer sizes are same now.
> 
> Signed-off-by: Imran Zaman <imran.zaman@intel.com>

Applied to u-boot/master, thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150915/cde46ec9/attachment.sig>