External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-12-22 21:55:25 +00:00
Code Issues Packages Projects Releases Wiki Activity

domain: Define explicit flags for saved image xml

Browse Source 
Commit d2a929d4 (0.9.4) defined virDomainSaveImageGetXMLDesc()'s use
of @flags as a subset of virDomainXMLFlags, documenting that 2 of the
3 flags defined at the time would never be valid.  Later, commit
28f8dfdc (1.0.0) introduced a new flag, VIR_DOMAIN_XML_MIGRATABLE, but
did not adjust the save image documentation to declare it as invalid.
Later, commit a67e3872 (3.7.0) blindly copied and pasted the same text
into virDomainManagedSaveGetXMLDesc.

However, since the flag is not accepted as valid by any of the
drivers (remote is just passthrough; and qemu is the only supporting
driver for either API, with support for just VIR_DOMAIN_XML_SECURE),
it is easier to just define an explicit set of supported flags
directly related to the save image API rather than trying to borrow
from live domain API, and risking confusion if even more domain flags
are added later (in fact, I have an upcoming patch that plans to add
a new flag to virDomainGetXMLDesc that makes no sense for saved
images).  We may someday decide that saved images need to support the
_MIGRATABLE flag, as it is possible to load a saved image with a
different version of libvirt than the one that created it, but that
can be a separate patch if it is ever needed.  Meanwhile, it DOES make
sense to reuse the same flags for SaveImage and for ManagedSave (since
ManagedSave is really just sugar for creating a normal SaveImage in a
location controlled by libvirt instead of by the user).

There is no API or ABI impact (since we purposefully used unsigned int
rather than an enum type in public API, and since the new flag name
carries the same value as the old reused name).

Signed-off-by: Eric Blake <eblake@redhat.com>
Reviewed-by: John Ferlan <jferlan@redhat.com>
This commit is contained in:
Eric Blake 2019-02-13 15:09:05 -06:00
parent 022eeddf29
commit 12a51f372a
4 changed files with 19 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/libvirt/libvirt-domain.h
View File

@ -1219,6 +1219,7 @@ int virDomainRestoreFlags (virConnectPtr conn,
const char *dxml, const char *dxml,
unsigned int flags); unsigned int flags);
/* See below for virDomainSaveImageXMLFlags */
char * virDomainSaveImageGetXMLDesc (virConnectPtr conn, char * virDomainSaveImageGetXMLDesc (virConnectPtr conn,
const char *file, const char *file,
unsigned int flags); unsigned int flags);
@ -1571,6 +1572,10 @@ typedef enum {
VIR_DOMAIN_XML_MIGRATABLE = (1 << 3), /* dump XML suitable for migration */ VIR_DOMAIN_XML_MIGRATABLE = (1 << 3), /* dump XML suitable for migration */
} virDomainXMLFlags; } virDomainXMLFlags;
typedef enum {
VIR_DOMAIN_SAVE_IMAGE_XML_SECURE = VIR_DOMAIN_XML_SECURE, /* dump security sensitive information too */
} virDomainSaveImageXMLFlags;
char * virDomainGetXMLDesc (virDomainPtr domain, char * virDomainGetXMLDesc (virDomainPtr domain,
unsigned int flags); unsigned int flags);

20
src/libvirt-domain.c
View File

@ -1066,16 +1066,15 @@ virDomainRestoreFlags(virConnectPtr conn, const char *from, const char *dxml,
* virDomainSaveImageGetXMLDesc: * virDomainSaveImageGetXMLDesc:
* @conn: pointer to the hypervisor connection * @conn: pointer to the hypervisor connection
* @file: path to saved state file * @file: path to saved state file
* @flags: bitwise-OR of subset of virDomainXMLFlags * @flags: bitwise-OR of supported virDomainSaveImageXMLFlags
* *
* This method will extract the XML describing the domain at the time * This method will extract the XML describing the domain at the time
* a saved state file was created. @file must be a file created * a saved state file was created. @file must be a file created
* previously by virDomainSave() or virDomainSaveFlags(). * previously by virDomainSave() or virDomainSaveFlags().
* *
* No security-sensitive data will be included unless @flags contains * No security-sensitive data will be included unless @flags contains
* VIR_DOMAIN_XML_SECURE; this flag is rejected on read-only * VIR_DOMAIN_SAVE_IMAGE_XML_SECURE; this flag is rejected on read-only
* connections. For this API, @flags should not contain either * connections.
* VIR_DOMAIN_XML_INACTIVE or VIR_DOMAIN_XML_UPDATE_CPU.
* *
* Returns a 0 terminated UTF-8 encoded XML instance, or NULL in case of * Returns a 0 terminated UTF-8 encoded XML instance, or NULL in case of
* error. The caller must free() the returned value. * error. The caller must free() the returned value.
@ -1092,7 +1091,8 @@ virDomainSaveImageGetXMLDesc(virConnectPtr conn, const char *file,
virCheckConnectReturn(conn, NULL); virCheckConnectReturn(conn, NULL);
virCheckNonNullArgGoto(file, error); virCheckNonNullArgGoto(file, error);
if ((conn->flags & VIR_CONNECT_RO) && (flags & VIR_DOMAIN_XML_SECURE)) { if ((conn->flags & VIR_CONNECT_RO) &&
(flags & VIR_DOMAIN_SAVE_IMAGE_XML_SECURE)) {
virReportError(VIR_ERR_OPERATION_DENIED, "%s", virReportError(VIR_ERR_OPERATION_DENIED, "%s",
_("virDomainSaveImageGetXMLDesc with secure flag")); _("virDomainSaveImageGetXMLDesc with secure flag"));
goto error; goto error;
@ -9477,15 +9477,14 @@ virDomainManagedSaveRemove(virDomainPtr dom, unsigned int flags)
/** /**
* virDomainManagedSaveGetXMLDesc: * virDomainManagedSaveGetXMLDesc:
* @domain: a domain object * @domain: a domain object
* @flags: bitwise-OR of subset of virDomainXMLFlags * @flags: bitwise-OR of supported virDomainSaveImageXMLFlags
* *
* This method will extract the XML description of the managed save * This method will extract the XML description of the managed save
* state file of a domain. * state file of a domain.
* *
* No security-sensitive data will be included unless @flags contains * No security-sensitive data will be included unless @flags contains
* VIR_DOMAIN_XML_SECURE; this flag is rejected on read-only * VIR_DOMAIN_SAVE_IMAGE_XML_SECURE; this flag is rejected on read-only
* connections. For this API, @flags should not contain either * connections.
* VIR_DOMAIN_XML_INACTIVE or VIR_DOMAIN_XML_UPDATE_CPU.
* *
* Returns a 0 terminated UTF-8 encoded XML instance, or NULL in case of * Returns a 0 terminated UTF-8 encoded XML instance, or NULL in case of
* error. The caller must free() the returned value. * error. The caller must free() the returned value.
@ -9502,7 +9501,8 @@ virDomainManagedSaveGetXMLDesc(virDomainPtr domain, unsigned int flags)
virCheckDomainReturn(domain, NULL); virCheckDomainReturn(domain, NULL);
conn = domain->conn; conn = domain->conn;
if ((conn->flags & VIR_CONNECT_RO) && (flags & VIR_DOMAIN_XML_SECURE)) { if ((conn->flags & VIR_CONNECT_RO) &&
(flags & VIR_DOMAIN_SAVE_IMAGE_XML_SECURE)) {
virReportError(VIR_ERR_OPERATION_DENIED, "%s", virReportError(VIR_ERR_OPERATION_DENIED, "%s",
_("virDomainManagedSaveGetXMLDesc with secure flag")); _("virDomainManagedSaveGetXMLDesc with secure flag"));
goto error; goto error;

6
src/qemu/qemu_driver.c
View File

@ -7078,8 +7078,7 @@ qemuDomainSaveImageGetXMLDesc(virConnectPtr conn, const char *path,
int fd = -1; int fd = -1;
virQEMUSaveDataPtr data = NULL; virQEMUSaveDataPtr data = NULL;
/* We only take subset of virDomainDefFormat flags. */ virCheckFlags(VIR_DOMAIN_SAVE_IMAGE_XML_SECURE, NULL);
virCheckFlags(VIR_DOMAIN_XML_SECURE, NULL);
fd = qemuDomainSaveImageOpen(driver, path, &def, &data, fd = qemuDomainSaveImageOpen(driver, path, &def, &data,
false, NULL, false, false); false, NULL, false, false);
@ -7183,8 +7182,7 @@ qemuDomainManagedSaveGetXMLDesc(virDomainPtr dom, unsigned int flags)
int fd = -1; int fd = -1;
virQEMUSaveDataPtr data = NULL; virQEMUSaveDataPtr data = NULL;
/* We only take subset of virDomainDefFormat flags. */ virCheckFlags(VIR_DOMAIN_SAVE_IMAGE_XML_SECURE, NULL);
virCheckFlags(VIR_DOMAIN_XML_SECURE, NULL);
if (!(vm = qemuDomObjFromDomain(dom))) if (!(vm = qemuDomObjFromDomain(dom)))
return ret; return ret;

4
src/remote/remote_protocol.x
View File

@ -5235,7 +5235,7 @@ enum remote_procedure {
* @generate: both * @generate: both
* @priority: high * @priority: high
* @acl: domain:read * @acl: domain:read
* @acl: domain:read_secure:VIR_DOMAIN_XML_SECURE * @acl: domain:read_secure:VIR_DOMAIN_SAVE_IMAGE_XML_SECURE
*/ */
REMOTE_PROC_DOMAIN_SAVE_IMAGE_GET_XML_DESC = 235, REMOTE_PROC_DOMAIN_SAVE_IMAGE_GET_XML_DESC = 235,
@ -6230,7 +6230,7 @@ enum remote_procedure {
/** /**
* @generate: both * @generate: both
* @acl: domain:read * @acl: domain:read
* @acl: domain:read_secure:VIR_DOMAIN_XML_SECURE * @acl: domain:read_secure:VIR_DOMAIN_SAVE_IMAGE_XML_SECURE
*/ */
REMOTE_PROC_DOMAIN_MANAGED_SAVE_GET_XML_DESC = 388, REMOTE_PROC_DOMAIN_MANAGED_SAVE_GET_XML_DESC = 388,