[libvirt] [PATCH v2 1/3] test: Use virDomainObjListFindByUUIDRef
Rather than using virDomainObjListFindByUUID, let's be more consistent
and return a reffed and locked object. Since we're using the Ref API,
use virDomainObjEndAPI on @dom and not just virObjectUnlock.
Signed-off-by: John Ferlan <jferlan(a)redhat.com>
---
src/test/test_driver.c | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 99c27cc0a0..0cacab2fde 100644
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -1740,16 +1740,14 @@ static virDomainPtr testDomainLookupByUUID(virConnectPtr conn,
virDomainPtr ret = NULL;
virDomainObjPtr dom;
- if (!(dom = virDomainObjListFindByUUID(privconn->domains, uuid))) {
+ if (!(dom = virDomainObjListFindByUUIDRef(privconn->domains, uuid))) {
virReportError(VIR_ERR_NO_DOMAIN, NULL);
- goto cleanup;
+ return NULL;
}
ret = virGetDomain(conn, dom->def->name, dom->def->uuid,
dom->def->id);
- cleanup:
- if (dom)
- virObjectUnlock(dom);
+ virDomainObjEndAPI(&dom);
return ret;
}
--
2.13.6