From a95b67bec3a4436970f4efe2e75c1256dc973841 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Tue, 28 May 2019 15:47:12 +0200 Subject: [PATCH] virDomainObjListAddLocked: Drop useless @cleanup label It's a premature optimization. It's perfectly acceptable for 'error' label to deal with @vm == NULL case. Signed-off-by: Michal Privoznik Reviewed-by: Nikolay Shirokovskiy --- src/conf/virdomainobjlist.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/conf/virdomainobjlist.c b/src/conf/virdomainobjlist.c index f5a7ecc4a4..d58d25f847 100644 --- a/src/conf/virdomainobjlist.c +++ b/src/conf/virdomainobjlist.c @@ -332,7 +332,7 @@ virDomainObjListAddLocked(virDomainObjListPtr doms, } if (!(vm = virDomainObjNew(xmlopt))) - goto cleanup; + goto error; vm->def = def; if (virDomainObjListAddObjLocked(doms, vm) < 0) { @@ -340,7 +340,7 @@ virDomainObjListAddLocked(virDomainObjListPtr doms, goto error; } } - cleanup: + return vm; error: -- 2.47.2