From 2e8ff1a22a54739c6cb03cf014779f389721e23d Mon Sep 17 00:00:00 2001 From: Ruslan Kuprieiev Date: Mon, 2 Dec 2013 19:50:44 +0400 Subject: [PATCH] v2 service: don't send dump response in case of self-dump without leave_running Currently we have a bug: if no leave_running is set in request, service won't send dump response. We must not send response only if it was a self-dump request and no leave_running option was set. Signed-off-by: Ruslan Kuprieiev Signed-off-by: Pavel Emelyanov --- cr-service.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/cr-service.c b/cr-service.c index eba5342a2..b09294849 100644 --- a/cr-service.c +++ b/cr-service.c @@ -166,6 +166,7 @@ static int setup_opts_from_req(int sk, CriuOpts *req) static int dump_using_req(int sk, CriuOpts *req) { bool success = false; + bool self_dump = !req->pid; if (setup_opts_from_req(sk, req) == -1) { pr_perror("Arguments treating fail"); @@ -175,7 +176,7 @@ static int dump_using_req(int sk, CriuOpts *req) if (cr_dump_tasks(req->pid) == -1) goto exit; - if (req->has_leave_running && req->leave_running) { + if (req->leave_running || !self_dump) { success = true; exit: if (send_criu_dump_resp(sk, success, false) == -1) {