On Thu, Apr 06, 2017 at 04:03:28PM -0500, Eric Blake wrote: > On 04/06/2017 03:59 PM, Richard W.M. Jones wrote: > > On Tue, Feb 28, 2017 at 09:36:42PM +0100, Kevin Wolf wrote: > >> This adds assertions that ensure that the necessary write permissions > >> have been granted before someone attempts to write to a node. > >> > >> Signed-off-by: Kevin Wolf <kw...@redhat.com> > >> Acked-by: Fam Zheng <f...@redhat.com> > >> Reviewed-by: Max Reitz <mre...@redhat.com> > >> --- > >> block/io.c | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >> diff --git a/block/io.c b/block/io.c > >> index 2592ca1..4c79745 100644 > >> --- a/block/io.c > >> +++ b/block/io.c > >> @@ -945,6 +945,8 @@ static int coroutine_fn > >> bdrv_co_do_copy_on_readv(BdrvChild *child, > >> size_t skip_bytes; > >> int ret; > >> > >> + assert(child->perm & (BLK_PERM_WRITE_UNCHANGED | BLK_PERM_WRITE)); > > > > > > This assertion is thrown in the libguestfs test suite. I filed a bug > > about it against the Fedora package: > > > > https://lists.gnu.org/archive/html/qemu-block/2017-02/msg01305.html > > Wrong URL?
Ooops, the right link is: https://bugzilla.redhat.com/show_bug.cgi?id=1439922 There is also a minimal reproducer in comment 2. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wordpress.com virt-df lists disk usage of guests without needing to install any software inside the virtual machine. Supports Linux and Windows. http://people.redhat.com/~rjones/virt-df/