From 504c205a0d4a790dba2be0dc0aab8b8d1b905a7d Mon Sep 17 00:00:00 2001 From: "Denis V. Lunev" Date: Wed, 5 Apr 2017 18:18:25 +0300 Subject: 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 CC: Kevin Wolf CC: Max Reitz Message-id: 1491405505-31620-3-git-send-email-den@openvz.org Reviewed-by: Eric Blake Signed-off-by: Max Reitz --- block.c | 2 ++ 1 file changed, 2 insertions(+) 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); -- cgit v1.2.3