提交 145dc7dd 编写于 作者: J John Ferlan

conf: Use VIR_AUTOUNREF in domain_conf

Let's make use of the auto __cleanup capabilities for virObjectUnref
consumers.
Signed-off-by: NJohn Ferlan <jferlan@redhat.com>
上级 c9b1b443
...@@ -1587,8 +1587,8 @@ void virDomainLeaseDefFree(virDomainLeaseDefPtr def) ...@@ -1587,8 +1587,8 @@ void virDomainLeaseDefFree(virDomainLeaseDefPtr def)
static virDomainVcpuDefPtr static virDomainVcpuDefPtr
virDomainVcpuDefNew(virDomainXMLOptionPtr xmlopt) virDomainVcpuDefNew(virDomainXMLOptionPtr xmlopt)
{ {
virObjectPtr priv = NULL;
virDomainVcpuDefPtr ret = NULL; virDomainVcpuDefPtr ret = NULL;
VIR_AUTOUNREF(virObjectPtr) priv = NULL;
if (xmlopt && xmlopt->privateData.vcpuNew && if (xmlopt && xmlopt->privateData.vcpuNew &&
!(priv = xmlopt->privateData.vcpuNew())) !(priv = xmlopt->privateData.vcpuNew()))
...@@ -1600,7 +1600,6 @@ virDomainVcpuDefNew(virDomainXMLOptionPtr xmlopt) ...@@ -1600,7 +1600,6 @@ virDomainVcpuDefNew(virDomainXMLOptionPtr xmlopt)
VIR_STEAL_PTR(ret->privateData, priv); VIR_STEAL_PTR(ret->privateData, priv);
cleanup: cleanup:
virObjectUnref(priv);
return ret; return ret;
} }
...@@ -19142,13 +19141,13 @@ virDomainCachetuneDefParse(virDomainDefPtr def, ...@@ -19142,13 +19141,13 @@ virDomainCachetuneDefParse(virDomainDefPtr def,
unsigned int flags) unsigned int flags)
{ {
xmlNodePtr oldnode = ctxt->node; xmlNodePtr oldnode = ctxt->node;
virResctrlAllocPtr alloc = NULL;
virDomainResctrlDefPtr resctrl = NULL; virDomainResctrlDefPtr resctrl = NULL;
ssize_t i = 0; ssize_t i = 0;
int n; int n;
int ret = -1; int ret = -1;
VIR_AUTOPTR(virBitmap) vcpus = NULL; VIR_AUTOPTR(virBitmap) vcpus = NULL;
VIR_AUTOFREE(xmlNodePtr *) nodes = NULL; VIR_AUTOFREE(xmlNodePtr *) nodes = NULL;
VIR_AUTOUNREF(virResctrlAllocPtr) alloc = NULL;
ctxt->node = node; ctxt->node = node;
...@@ -19207,7 +19206,6 @@ virDomainCachetuneDefParse(virDomainDefPtr def, ...@@ -19207,7 +19206,6 @@ virDomainCachetuneDefParse(virDomainDefPtr def,
cleanup: cleanup:
ctxt->node = oldnode; ctxt->node = oldnode;
virDomainResctrlDefFree(resctrl); virDomainResctrlDefFree(resctrl);
virObjectUnref(alloc);
return ret; return ret;
} }
...@@ -19353,10 +19351,10 @@ virDomainMemorytuneDefParse(virDomainDefPtr def, ...@@ -19353,10 +19351,10 @@ virDomainMemorytuneDefParse(virDomainDefPtr def,
unsigned int flags) unsigned int flags)
{ {
xmlNodePtr oldnode = ctxt->node; xmlNodePtr oldnode = ctxt->node;
virResctrlAllocPtr alloc = NULL;
virDomainResctrlDefPtr resctrl = NULL; virDomainResctrlDefPtr resctrl = NULL;
VIR_AUTOPTR(virBitmap) vcpus = NULL; VIR_AUTOPTR(virBitmap) vcpus = NULL;
VIR_AUTOFREE(xmlNodePtr *) nodes = NULL; VIR_AUTOFREE(xmlNodePtr *) nodes = NULL;
VIR_AUTOUNREF(virResctrlAllocPtr) alloc = NULL;
ssize_t i = 0; ssize_t i = 0;
int n; int n;
...@@ -19414,7 +19412,6 @@ virDomainMemorytuneDefParse(virDomainDefPtr def, ...@@ -19414,7 +19412,6 @@ virDomainMemorytuneDefParse(virDomainDefPtr def,
cleanup: cleanup:
ctxt->node = oldnode; ctxt->node = oldnode;
virDomainResctrlDefFree(resctrl); virDomainResctrlDefFree(resctrl);
virObjectUnref(alloc);
return ret; return ret;
} }
...@@ -30460,11 +30457,11 @@ virDomainNetBandwidthUpdate(virDomainNetDefPtr iface, ...@@ -30460,11 +30457,11 @@ virDomainNetBandwidthUpdate(virDomainNetDefPtr iface,
int int
virDomainNetResolveActualType(virDomainNetDefPtr iface) virDomainNetResolveActualType(virDomainNetDefPtr iface)
{ {
virConnectPtr conn = NULL;
virNetworkPtr net = NULL;
virNetworkDefPtr def = NULL; virNetworkDefPtr def = NULL;
int ret = -1; int ret = -1;
VIR_AUTOFREE(char *) xml = NULL; VIR_AUTOFREE(char *) xml = NULL;
VIR_AUTOUNREF(virConnectPtr) conn = NULL;
VIR_AUTOUNREF(virNetworkPtr) net = NULL;
if (iface->type != VIR_DOMAIN_NET_TYPE_NETWORK) if (iface->type != VIR_DOMAIN_NET_TYPE_NETWORK)
return iface->type; return iface->type;
...@@ -30531,8 +30528,6 @@ virDomainNetResolveActualType(virDomainNetDefPtr iface) ...@@ -30531,8 +30528,6 @@ virDomainNetResolveActualType(virDomainNetDefPtr iface)
cleanup: cleanup:
virNetworkDefFree(def); virNetworkDefFree(def);
virObjectUnref(conn);
virObjectUnref(net);
return ret; return ret;
} }
...@@ -30656,13 +30651,13 @@ virDomainDiskTranslateISCSIDirect(virDomainDiskDefPtr def, ...@@ -30656,13 +30651,13 @@ virDomainDiskTranslateISCSIDirect(virDomainDiskDefPtr def,
int int
virDomainDiskTranslateSourcePool(virDomainDiskDefPtr def) virDomainDiskTranslateSourcePool(virDomainDiskDefPtr def)
{ {
virConnectPtr conn = NULL;
virStoragePoolPtr pool = NULL;
virStorageVolPtr vol = NULL;
virStorageVolInfo info; virStorageVolInfo info;
int ret = -1; int ret = -1;
VIR_AUTOPTR(virStoragePoolDef) pooldef = NULL; VIR_AUTOPTR(virStoragePoolDef) pooldef = NULL;
VIR_AUTOFREE(char *) poolxml = NULL; VIR_AUTOFREE(char *) poolxml = NULL;
VIR_AUTOUNREF(virConnectPtr) conn = NULL;
VIR_AUTOUNREF(virStoragePoolPtr) pool = NULL;
VIR_AUTOUNREF(virStorageVolPtr) vol = NULL;
if (def->src->type != VIR_STORAGE_TYPE_VOLUME) if (def->src->type != VIR_STORAGE_TYPE_VOLUME)
return 0; return 0;
...@@ -30817,9 +30812,6 @@ virDomainDiskTranslateSourcePool(virDomainDiskDefPtr def) ...@@ -30817,9 +30812,6 @@ virDomainDiskTranslateSourcePool(virDomainDiskDefPtr def)
ret = 0; ret = 0;
cleanup: cleanup:
virObjectUnref(conn);
virObjectUnref(pool);
virObjectUnref(vol);
return ret; return ret;
} }
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册