From 62c677849509728419ba700507be6a8ec38a39f7 Mon Sep 17 00:00:00 2001 From: John Ferlan Date: Tue, 1 Sep 2015 07:20:15 -0400 Subject: [PATCH] conf: Remove need for a couple of sa_asserts Remove the need for a couple of sa_asserts. --- src/conf/domain_conf.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c index f95190ff83..6df1618d75 100644 --- a/src/conf/domain_conf.c +++ b/src/conf/domain_conf.c @@ -24067,10 +24067,11 @@ virDomainObjListCollect(virDomainObjListPtr domlist, unsigned int flags) { struct virDomainListData data = { NULL, 0 }; + ssize_t hash_size; virObjectLock(domlist); - sa_assert(domlist->objs); - if (VIR_ALLOC_N(data.vms, virHashSize(domlist->objs)) < 0) { + if ((hash_size = virHashSize(domlist->objs)) < 0 || + (VIR_ALLOC_N(data.vms, hash_size) < 0)) { virObjectUnlock(domlist); return -1; } @@ -24132,8 +24133,8 @@ virDomainObjListConvert(virDomainObjListPtr domlist, } virObjectUnlock(domlist); - sa_assert(*vms); - virDomainObjListFilter(vms, nvms, conn, filter, flags); + if (*vms) + virDomainObjListFilter(vms, nvms, conn, filter, flags); return 0; -- GitLab