summaryrefslogtreecommitdiff
path: root/block.c
diff options
context:
space:
mode:
authorDenis V. Lunev <den@openvz.org>2017-04-05 18:18:25 +0300
committerMax Reitz <mreitz@redhat.com>2017-04-28 16:02:03 +0200
commit504c205a0d4a790dba2be0dc0aab8b8d1b905a7d (patch)
tree41f3eb620d439d0333426c7b0c21de8382e5ba3e /block.c
parentd4a7f45ec9a54ca52e6381a792dc2aaea656338a (diff)
downloadqemu-504c205a0d4a790dba2be0dc0aab8b8d1b905a7d.tar.gz
block: assert no image modification under BDRV_O_INACTIVE
As long as BDRV_O_INACTIVE is set, the image file is only opened so we have a file descriptor for it. We're definitely not supposed to modify the image, it's still owned by the migration source. This commit is an addition to 09e0c771 but the assert() is added to bdrv_truncate(). Signed-off-by: Denis V. Lunev <den@openvz.org> CC: Kevin Wolf <kwolf@redhat.com> CC: Max Reitz <mreitz@redhat.com> Message-id: 1491405505-31620-3-git-send-email-den@openvz.org Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Max Reitz <mreitz@redhat.com>
Diffstat (limited to 'block.c')
-rw-r--r--block.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/block.c b/block.c
index 76bf00f4b2..6c6bb3ec7a 100644
--- a/block.c
+++ b/block.c
@@ -3328,6 +3328,8 @@ int bdrv_truncate(BdrvChild *child, int64_t offset, Error **errp)
return -EACCES;
}
+ assert(!(bs->open_flags & BDRV_O_INACTIVE));
+
ret = drv->bdrv_truncate(bs, offset, errp);
if (ret == 0) {
ret = refresh_total_sectors(bs, offset >> BDRV_SECTOR_BITS);