commit e140f4b7b8c0fdda866ba3d8b4aa184b6ba6d6b8
author Lukas Straub <lukasstraub2@web.de> Mon May 11 09:08:01 2020 +0200
committer Kevin Wolf <kwolf@redhat.com> Mon May 18 19:05:25 2020 +0200
tree 9b72a7f2703fa404d4a63cac58fa69bdd1aaa4c1
parent 16cea4ee1c8e5a69a058e76f426b2e17974d8d7d

block/replication.c: Avoid cancelling the job twice

If qemu in colo secondary mode is stopped, it crashes because
s->backup_job is canceled twice: First with job_cancel_sync_all()
in qemu_cleanup() and then in replication_stop().

Fix this by assigning NULL to s->backup_job when the job completes
so replication_stop() and replication_do_checkpoint() won't touch
the job.

Signed-off-by: Lukas Straub <lukasstraub2@web.de>
Message-Id: <20200511090801.7ed5d8f3@luklap>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>

block/replication.c

diff --git a/block/replication.c b/block/replication.c
index 971f0fe..c03980a 100644
--- a/block/replication.c
+++ b/block/replication.c
@@ -398,6 +398,8 @@
     BDRVReplicationState *s = bs->opaque;
     BlockDriverState *top_bs;
 
+    s->backup_job = NULL;
+
     top_bs = bdrv_lookup_bs(s->top_id, s->top_id, NULL);
     if (!top_bs) {
         return;