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

Andrei Vagin avagin at virtuozzo.com
Fri Mar 31 14:16:58 PDT 2017


Applied

On Wed, Mar 29, 2017 at 11:15:00PM +0300, Andrei Vagin wrote:
> 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