---
src/libxl/libxl_conf.c | 18 +++++-------------
src/libxl/libxl_driver.c | 46 +++++++++++-----------------------------------
2 files changed, 16 insertions(+), 48 deletions(-)
diff --git a/src/libxl/libxl_conf.c b/src/libxl/libxl_conf.c
index e170357..1b27b6f 100644
--- a/src/libxl/libxl_conf.c
+++ b/src/libxl/libxl_conf.c
@@ -278,12 +278,11 @@ libxlMakeCapabilitiesInternal(virArch hostarch,
host_pae,
guest_archs,
nr_guest_archs)) == NULL)
- goto no_memory;
+ goto error;
return caps;
- no_memory:
- virReportOOMError();
+ error:
virObjectUnref(caps);
return NULL;
}
@@ -576,10 +575,8 @@ libxlMakeDiskList(virDomainDefPtr def, libxl_domain_config *d_config)
libxl_device_disk *x_disks;
int i;
- if (VIR_ALLOC_N(x_disks, ndisks) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(x_disks, ndisks) < 0)
return -1;
- }
for (i = 0; i < ndisks; i++) {
if (libxlMakeDisk(l_disks[i], &x_disks[i]) < 0)
@@ -648,10 +645,8 @@ libxlMakeNicList(virDomainDefPtr def, libxl_domain_config *d_config)
libxl_device_nic *x_nics;
int i;
- if (VIR_ALLOC_N(x_nics, nnics) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(x_nics, nnics) < 0)
return -1;
- }
for (i = 0; i < nnics; i++) {
if (libxlMakeNic(l_nics[i], &x_nics[i]))
@@ -734,12 +729,9 @@ libxlMakeVfbList(libxlDriverPrivatePtr driver,
if (nvfbs == 0)
return 0;
- if (VIR_ALLOC_N(x_vfbs, nvfbs) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(x_vfbs, nvfbs) < 0)
return -1;
- }
if (VIR_ALLOC_N(x_vkbs, nvfbs) < 0) {
- virReportOOMError();
VIR_FREE(x_vfbs);
return -1;
}
diff --git a/src/libxl/libxl_driver.c b/src/libxl/libxl_driver.c
index 1bae3d6..c6f9616 100644
--- a/src/libxl/libxl_driver.c
+++ b/src/libxl/libxl_driver.c
@@ -183,10 +183,8 @@ libxlFDRegisterEventHook(void *priv, int fd, void **hndp,
int vir_events = VIR_EVENT_HANDLE_ERROR;
libxlEventHookInfoPtr info;
- if (VIR_ALLOC(info) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC(info) < 0)
return -1;
- }
info->priv = priv;
/*
@@ -287,10 +285,8 @@ libxlTimeoutRegisterEventHook(void *priv,
static struct timeval zero;
int timeout;
- if (VIR_ALLOC(info) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC(info) < 0)
return -1;
- }
info->priv = priv;
/*
@@ -548,11 +544,7 @@ static char *
libxlDomainManagedSavePath(libxlDriverPrivatePtr driver, virDomainObjPtr vm) {
char *ret;
- if (virAsprintf(&ret, "%s/%s.save", driver->saveDir, vm->def->name) < 0) {
- virReportOOMError();
- return NULL;
- }
-
+ ignore_value(virAsprintf(&ret, "%s/%s.save", driver->saveDir, vm->def->name));
return ret;
}
@@ -597,10 +589,8 @@ libxlSaveImageOpen(libxlDriverPrivatePtr driver, const char *from,
goto error;
}
- if (VIR_ALLOC_N(xml, hdr.xmlLen) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(xml, hdr.xmlLen) < 0)
goto error;
- }
if (saferead(fd, xml, hdr.xmlLen) != hdr.xmlLen) {
virReportError(VIR_ERR_OPERATION_FAILED, "%s", _("failed to read XML"));
@@ -845,10 +835,8 @@ libxlDomainSetVcpuAffinities(libxlDriverPrivatePtr driver, virDomainObjPtr vm)
if (vcpu != def->cputune.vcpupin[vcpu]->vcpuid)
continue;
- if (VIR_ALLOC_N(cpumap, cpumaplen) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(cpumap, cpumaplen) < 0)
goto cleanup;
- }
cpumask = (uint8_t*) def->cputune.vcpupin[vcpu]->cpumask;
@@ -1274,7 +1262,7 @@ libxlStateInitialize(bool privileged,
}
if (virAsprintf(&log_file, "%s/libxl.log", libxl_driver->logDir) < 0) {
- goto out_of_memory;
+ goto error;
}
if ((libxl_driver->logger_file = fopen(log_file, "a")) == NULL) {
@@ -1369,8 +1357,6 @@ libxlStateInitialize(bool privileged,
return 0;
-out_of_memory:
- virReportOOMError();
error:
ret = -1;
fail:
@@ -2803,10 +2789,8 @@ libxlDomainSetVcpusFlags(virDomainPtr dom, unsigned int nvcpus,
goto cleanup;
maplen = VIR_CPU_MAPLEN(nvcpus);
- if (VIR_ALLOC_N(bitmask, maplen) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(bitmask, maplen) < 0)
goto cleanup;
- }
for (i = 0; i < nvcpus; ++i) {
pos = i / 8;
@@ -2967,10 +2951,8 @@ libxlDomainPinVcpu(virDomainPtr dom, unsigned int vcpu, unsigned char *cpumap,
}
if (!vm->def->cputune.vcpupin) {
- if (VIR_ALLOC(vm->def->cputune.vcpupin) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC(vm->def->cputune.vcpupin) < 0)
goto cleanup;
- }
vm->def->cputune.nvcpupin = 0;
}
if (virDomainVcpuPinAdd(&vm->def->cputune.vcpupin,
@@ -3176,10 +3158,8 @@ libxlConnectDomainXMLToNative(virConnectPtr conn, const char * nativeFormat,
if (!(conf = xenFormatXM(conn, def, ver_info->xen_version_major)))
goto cleanup;
- if (VIR_ALLOC_N(ret, len) < 0) {
- virReportOOMError();
+ if (VIR_ALLOC_N(ret, len) < 0)
goto cleanup;
- }
if (virConfWriteMem(ret, &len, conf) < 0) {
VIR_FREE(ret);
@@ -3491,10 +3471,8 @@ libxlDomainAttachDeviceDiskLive(libxlDomainObjPrivatePtr priv,
goto cleanup;
}
- if (VIR_REALLOC_N(vm->def->disks, vm->def->ndisks+1) < 0) {
- virReportOOMError();
+ if (VIR_REALLOC_N(vm->def->disks, vm->def->ndisks+1) < 0)
goto cleanup;
- }
if (libxlMakeDisk(l_disk, &x_disk) < 0)
goto cleanup;
@@ -3616,10 +3594,8 @@ libxlDomainAttachDeviceConfig(virDomainDefPtr vmdef, virDomainDeviceDefPtr dev)
_("target %s already exists."), disk->dst);
return -1;
}
- if (virDomainDiskInsert(vmdef, disk)) {
- virReportOOMError();
+ if (virDomainDiskInsert(vmdef, disk))
return -1;
- }
/* vmdef has the pointer. Generic codes for vmdef will do all jobs */
dev->data.disk = NULL;
break;
--
1.8.1.5
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list