[libvirt] [PATCH] virsh: blockcopy: force an absolute path

[Martin Kletzander]

On Wed, Apr 20, 2016 at 01:33:15PM -0400, Cole Robinson wrote:
>virsh doesn't reject or absolutify a passed relative path, which
>can give unexpected results. Convert a relative path to an absolute
>one before calling the API
>
>https://bugzilla.redhat.com/show_bug.cgi?id=1300177
>---
> tools/virsh-domain.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>

This is a nasty bug, that's for sure.  However, shouldn't our API be
fixed rather than just virsh?  There's still a bug there.

Moreover, IIUC the path you give it should be relative to the volume (or
something specific) and not the path you're calling it with.  Well, it
could make sense to make it relative to CWD, but what if it's called
from a remote client?  That path might not exist at all.  I think it
should be relative to something else and the volume seems the most
sensible (after thinking about it for roughly 14 seconds).

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20160421/f33f7fa2/attachment-0001.sig>