]> git.proxmox.com Git - mirror_qemu.git/commitdiff
block/backup: drop dead code from backup_job_create
authorVladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Thu, 17 Oct 2019 14:21:22 +0000 (17:21 +0300)
committerKevin Wolf <kwolf@redhat.com>
Fri, 25 Oct 2019 13:15:01 +0000 (15:15 +0200)
After commit 00e30f05de1d195, there is no more "goto error" points
after job creation, so after "error:" @job is always NULL and we don't
need roll-back job creation.

Reported-by: Coverity (CID 1406402)
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
Acked-by: Stefano Garzarella <sgarzare@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
block/backup.c

index dddcf77f53024d33bbe76d6cd2fa00fae545dd85..cf62b1a38ceacaa6785f0ec0d1974fd11669016b 100644 (file)
@@ -474,10 +474,7 @@ BlockJob *backup_job_create(const char *job_id, BlockDriverState *bs,
     if (sync_bitmap) {
         bdrv_reclaim_dirty_bitmap(sync_bitmap, NULL);
     }
-    if (job) {
-        backup_clean(&job->common.job);
-        job_early_fail(&job->common.job);
-    } else if (backup_top) {
+    if (backup_top) {
         bdrv_backup_top_drop(backup_top);
     }