On Thursday, 3 August 2017 19:13:45 CEST Richard W.M. Jones wrote:
https://bugzilla.redhat.com/show_bug.cgi?id=1477623
The first two patches are cleanups.
The third patch changes the way that we handle Device and Dev_or_Path
parameters so that a parameter marked as such can really only contain
a block device name (and not, for instance, a chardev). Using a
chardev here caused hangs in the API.
The next two patches fix API usage to conform to this new stricter
checking. I removed the ability to use "/dev/urandom" (and similar)
non-block devices in place of block devices in such APIs as
guestfs_copy_device_to_device.
Last patch adds a regression test.
Except for the notes for patch #3, the rest LGTM.
--
Pino Toscano