[CRIU] [PATCH] service: set success to true before exit

Ruslan Kuprieiev kupruser at gmail.com
Thu Dec 5 03:03:54 PST 2013


This patch fixes bug, which makes criu report fail, if no need to send resp.

On 05.12.2013 15:01, Ruslan Kuprieiev wrote:
> Signed-off-by: Ruslan Kuprieiev <kupruser at gmail.com>
> ---
>   cr-service.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/cr-service.c b/cr-service.c
> index b092948..a82ef9e 100644
> --- a/cr-service.c
> +++ b/cr-service.c
> @@ -176,9 +176,9 @@ static int dump_using_req(int sk, CriuOpts *req)
>   	if (cr_dump_tasks(req->pid) == -1)
>   		goto exit;
>   
> -	if (req->leave_running || !self_dump) {
> -		success = true;
> +	success = true;
>   exit:
> +	if (req->leave_running  || !self_dump) {
>   		if (send_criu_dump_resp(sk, success, false) == -1) {
>   			pr_perror("Can't send response");
>   			success = false;



More information about the CRIU mailing list