[CRIU] [PATCH] zdtm/dumpable02: don't print a non-null terminated string

Andrei Vagin avagin at openvz.org
Wed Mar 29 13:15:00 PDT 2017


From: Andrei Vagin <avagin at virtuozzo.com>

Signed-off-by: Andrei Vagin <avagin at virtuozzo.com>
---
 test/zdtm/static/dumpable02.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/test/zdtm/static/dumpable02.c b/test/zdtm/static/dumpable02.c
index 3463ded..bd632e1 100644
--- a/test/zdtm/static/dumpable02.c
+++ b/test/zdtm/static/dumpable02.c
@@ -36,11 +36,12 @@ int get_dumpable_from_pipes(int pipe_input, int pipe_output) {
 	/* input and output are from the child's point of view. */
 
 	write(pipe_input, "GET\n", 4);
-	len = read(pipe_output, buf, sizeof(buf));
+	len = read(pipe_output, buf, sizeof(buf) - 1);
 	if (len < 0) {
 		pr_perror("error in parent reading from pipe");
 		return -1;
 	}
+	buf[len] = 0;
 
 	if (memcmp(buf, "DUMPABLE:", 9) != 0) {
 		pr_perror("child returned [%s]", buf);
-- 
2.7.4



More information about the CRIU mailing list