|
|
357786 |
From fb76c3aaa2e3d54b20fef63fcc64d2ff5a394b08 Mon Sep 17 00:00:00 2001
|
|
|
357786 |
From: John Snow <jsnow@redhat.com>
|
|
|
357786 |
Date: Wed, 18 Jul 2018 22:54:44 +0200
|
|
|
357786 |
Subject: [PATCH 59/89] block/qcow2-bitmap: fix free_bitmap_clusters
|
|
|
357786 |
|
|
|
357786 |
RH-Author: John Snow <jsnow@redhat.com>
|
|
|
357786 |
Message-id: <20180718225511.14878-9-jsnow@redhat.com>
|
|
|
357786 |
Patchwork-id: 81392
|
|
|
357786 |
O-Subject: [RHEL-7.6 qemu-kvm-rhev PATCH 08/35] block/qcow2-bitmap: fix free_bitmap_clusters
|
|
|
357786 |
Bugzilla: 1207657
|
|
|
357786 |
RH-Acked-by: Eric Blake <eblake@redhat.com>
|
|
|
357786 |
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
|
|
|
357786 |
RH-Acked-by: Fam Zheng <famz@redhat.com>
|
|
|
357786 |
|
|
|
357786 |
From: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
|
|
|
357786 |
|
|
|
357786 |
This assert may fail, because bitmap_table is not initialized. Just
|
|
|
357786 |
drop it, as it's obvious, that bitmap_table_load sets bitmap_table
|
|
|
357786 |
parameter only when returning zero.
|
|
|
357786 |
|
|
|
357786 |
Reported-by: Pavel Butsykin <pbutsykin@virtuozzo.com>
|
|
|
357786 |
Signed-off-by: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>
|
|
|
357786 |
Message-id: 20180608101225.2575-1-vsementsov@virtuozzo.com
|
|
|
357786 |
Signed-off-by: Max Reitz <mreitz@redhat.com>
|
|
|
357786 |
(cherry picked from commit 7eb24009dbf8102fc27d5459bec1cb8a932540c4)
|
|
|
357786 |
Signed-off-by: John Snow <jsnow@redhat.com>
|
|
|
357786 |
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
|
357786 |
---
|
|
|
357786 |
block/qcow2-bitmap.c | 1 -
|
|
|
357786 |
1 file changed, 1 deletion(-)
|
|
|
357786 |
|
|
|
357786 |
diff --git a/block/qcow2-bitmap.c b/block/qcow2-bitmap.c
|
|
|
357786 |
index 6e93ec4..3e4e4e4 100644
|
|
|
357786 |
--- a/block/qcow2-bitmap.c
|
|
|
357786 |
+++ b/block/qcow2-bitmap.c
|
|
|
357786 |
@@ -254,7 +254,6 @@ static int free_bitmap_clusters(BlockDriverState *bs, Qcow2BitmapTable *tb)
|
|
|
357786 |
|
|
|
357786 |
ret = bitmap_table_load(bs, tb, &bitmap_table);
|
|
|
357786 |
if (ret < 0) {
|
|
|
357786 |
- assert(bitmap_table == NULL);
|
|
|
357786 |
return ret;
|
|
|
357786 |
}
|
|
|
357786 |
|
|
|
357786 |
--
|
|
|
357786 |
1.8.3.1
|
|
|
357786 |
|