[dpdk-dev] [PATCH] Fix buffer overflow issue in app/test/commands.c(we add 1 extra byte for trailing \0).

Ming Zhao mzhao at luminatewireless.com
Fri Feb 26 08:11:11 CET 2016


Fix memleak of cmdline in app/test/test.c
---
 app/test/commands.c | 2 +-
 app/test/test.c     | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/app/test/commands.c b/app/test/commands.c
index 9cb9606..5dfa599 100644
--- a/app/test/commands.c
+++ b/app/test/commands.c
@@ -439,7 +439,7 @@ int commands_init(void)
 		commands_len += strlen(t->command) + 1;
 	}
 
-	commands = malloc(commands_len);
+	commands = malloc(commands_len + 1); /* one more byte for \0 */
 	if (!commands)
 		return -1;
 
diff --git a/app/test/test.c b/app/test/test.c
index f35b304..108a347 100644
--- a/app/test/test.c
+++ b/app/test/test.c
@@ -150,6 +150,7 @@ main(int argc, char **argv)
 	}
 	cmdline_interact(cl);
 	cmdline_stdin_exit(cl);
+        cmdline_free(cl);
 #endif
 
 	return 0;
-- 
1.9.1



More information about the dev mailing list