External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2025-01-22 20:45:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

conf: snapshot: rename variable holding memory snapshot file location

Browse Source 
'file' is too generic to know what's going on. Rename it to
'memorysnapshotfile'.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
This commit is contained in:
Peter Krempa 2021-06-16 16:54:18 +02:00
parent 308aafe289
commit b30a8ee67d
3 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/conf/snapshot_conf.c
View File

@ -125,7 +125,7 @@ virDomainSnapshotDefDispose(void *obj)
virDomainSnapshotDef *def = obj; virDomainSnapshotDef *def = obj;
size_t i; size_t i;
g_free(def->file); g_free(def->memorysnapshotfile);
for (i = 0; i < def->ndisks; i++) for (i = 0; i < def->ndisks; i++)
virDomainSnapshotDiskDefClear(&def->disks[i]); virDomainSnapshotDiskDefClear(&def->disks[i]);
g_free(def->disks); g_free(def->disks);
@ -360,13 +360,13 @@ virDomainSnapshotDefParse(xmlXPathContextPtr ctxt,
"disk-only snapshot")); "disk-only snapshot"));
goto cleanup; goto cleanup;
} }
def->file = g_steal_pointer(&memoryFile); def->memorysnapshotfile = g_steal_pointer(&memoryFile);
/* verify that memory path is absolute */ /* verify that memory path is absolute */
if (def->file && !g_path_is_absolute(def->file)) { if (def->memorysnapshotfile && !g_path_is_absolute(def->memorysnapshotfile)) {
virReportError(VIR_ERR_XML_ERROR, virReportError(VIR_ERR_XML_ERROR,
_("memory snapshot file path (%s) must be absolute"), _("memory snapshot file path (%s) must be absolute"),
def->file); def->memorysnapshotfile);
goto cleanup; goto cleanup;
} }
@ -863,7 +863,7 @@ virDomainSnapshotDefFormatInternal(virBuffer *buf,
if (def->memory) { if (def->memory) {
virBufferAsprintf(buf, "<memory snapshot='%s'", virBufferAsprintf(buf, "<memory snapshot='%s'",
virDomainSnapshotLocationTypeToString(def->memory)); virDomainSnapshotLocationTypeToString(def->memory));
virBufferEscapeString(buf, " file='%s'", def->file); virBufferEscapeString(buf, " file='%s'", def->memorysnapshotfile);
virBufferAddLit(buf, "/>\n"); virBufferAddLit(buf, "/>\n");
} }

2
src/conf/snapshot_conf.h
View File

@ -82,7 +82,7 @@ struct _virDomainSnapshotDef {
int state; /* virDomainSnapshotState */ int state; /* virDomainSnapshotState */
int memory; /* virDomainMemorySnapshot */ int memory; /* virDomainMemorySnapshot */
char *file; /* memory state file when snapshot is external */ char *memorysnapshotfile; /* memory state file when snapshot is external */
size_t ndisks; /* should not exceed dom->ndisks */ size_t ndisks; /* should not exceed dom->ndisks */
virDomainSnapshotDiskDef *disks; virDomainSnapshotDiskDef *disks;

8
src/qemu/qemu_snapshot.c
View File

@ -1451,9 +1451,9 @@ qemuSnapshotCreateActiveExternal(virQEMUDriver *driver,
goto cleanup; goto cleanup;
xml = NULL; xml = NULL;
if ((ret = qemuSaveImageCreate(driver, vm, snapdef->file, data, if ((ret = qemuSaveImageCreate(driver, vm, snapdef->memorysnapshotfile,
compressor, 0, data, compressor, 0,
QEMU_ASYNC_JOB_SNAPSHOT)) < 0) QEMU_ASYNC_JOB_SNAPSHOT)) < 0)
goto cleanup; goto cleanup;
/* the memory image was created, remove it on errors */ /* the memory image was created, remove it on errors */
@ -1522,7 +1522,7 @@ qemuSnapshotCreateActiveExternal(virQEMUDriver *driver,
virQEMUSaveDataFree(data); virQEMUSaveDataFree(data);
if (memory_unlink && ret < 0) if (memory_unlink && ret < 0)
unlink(snapdef->file); unlink(snapdef->memorysnapshotfile);
return ret; return ret;
} }