External/libvirt
1
0
Fork 0
Code

virNodeDevCapsDefParseXML: Use g_autoptr

Browse Source 
Signed-off-by: Tim Wiederhake <twiederh@redhat.com>
Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
This commit is contained in:
Tim Wiederhake 2021-05-10 14:48:40 +02:00 committed by Michal Privoznik
parent 27b09ff09f
commit 1f59f2fd3d
1 changed files with 4 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/conf/node_device_conf.c
View File

@ -1971,14 +1971,12 @@ virNodeDevCapsDefParseXML(xmlXPathContextPtr ctxt,
int create, int create,
const char *virt_type) const char *virt_type)
{ {
virNodeDevCapsDef *caps; g_autoptr(virNodeDevCapsDef) caps = g_new0(virNodeDevCapsDef, 1);
int ret = -1; int ret = -1;
caps = g_new0(virNodeDevCapsDef, 1);
if (virXMLPropEnum(node, "type", virNodeDevCapTypeFromString, if (virXMLPropEnum(node, "type", virNodeDevCapTypeFromString,
VIR_XML_PROP_REQUIRED, &caps->data.type) < 0) VIR_XML_PROP_REQUIRED, &caps->data.type) < 0)
goto error; return NULL;
switch (caps->data.type) { switch (caps->data.type) {
case VIR_NODE_DEV_CAP_SYSTEM: case VIR_NODE_DEV_CAP_SYSTEM:
@ -2050,12 +2048,9 @@ virNodeDevCapsDefParseXML(xmlXPathContextPtr ctxt,
} }
if (ret < 0) if (ret < 0)
goto error;
return caps;
error:
virNodeDevCapsDefFree(caps);
return NULL; return NULL;
return g_steal_pointer(&caps);
} }