[libvirt] [PATCH] domain: avoid slash characters to the new domain name.

Peter Krempa pkrempa at redhat.com
Tue Apr 17 07:14:59 UTC 2018


On Tue, Apr 17, 2018 at 00:27:27 -0300, Julio Faracco wrote:
> The 'domrename' command needs to check if the new domain name contains
> the slash character. This character is not accepted by libvirt XML
> definition because it is an invalid char (see Cole's commit b1fc6a7b7).
> This commit enhace the 'domrename' command adding this check.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1333232
> 
> Signed-off-by: Julio Faracco <jcfaracco at gmail.com>
> ---
>  src/internal.h       | 9 +++++++++
>  src/libvirt-domain.c | 1 +
>  2 files changed, 10 insertions(+)

Slash in VM name may be allowed for some hypervisors and when defining
the hypervisor defines this via VIR_DOMAIN_DEF_FEATURE_NAME_SLASH parser
feature flag.

Currently it appears that virtualbox, phyp, vmx, and xenapi driver
support a slash so the fix below is wrong as it would forbid the slash
when renaming the VM for those drivers.

This needs to be implemented in the qemu driver according to the BZ.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20180417/29cbdaab/attachment-0001.sig>


More information about the libvir-list mailing list