]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
virDomainSEVDefParseXML: Remove superfluous `goto`s
authorTim Wiederhake <twiederh@redhat.com>
Mon, 5 Jul 2021 10:46:53 +0000 (12:46 +0200)
committerJán Tomko <jtomko@redhat.com>
Wed, 7 Jul 2021 13:34:12 +0000 (15:34 +0200)
Signed-off-by: Tim Wiederhake <twiederh@redhat.com>
Reviewed-by: Ján Tomko <jtomko@redhat.com>
Signed-off-by: Ján Tomko <jtomko@redhat.com>
src/conf/domain_conf.c

index dd803e6df5a2298520b19c10e2cdbc98c6b6721e..db8ec23d700288a73b9f51c8a293ed788ae30d82 100644 (file)
@@ -14730,12 +14730,12 @@ virDomainSEVDefParseXML(xmlNodePtr sevNode,
     if (virXMLPropEnum(sevNode, "type", virDomainLaunchSecurityTypeFromString,
                        VIR_XML_PROP_NONZERO | VIR_XML_PROP_REQUIRED,
                        &def->sectype) < 0)
-        goto error;
+        return NULL;
 
     if (virXPathULongHex("string(./policy)", ctxt, &policy) < 0) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("failed to get launch security policy"));
-        goto error;
+        return NULL;
     }
 
     /* the following attributes are platform dependent and if missing, we can
@@ -14747,7 +14747,7 @@ virDomainSEVDefParseXML(xmlNodePtr sevNode,
     } else if (rc == -2) {
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("Invalid format for launch security cbitpos"));
-        goto error;
+        return NULL;
     }
 
     rc = virXPathUInt("string(./reducedPhysBits)", ctxt,
@@ -14758,7 +14758,7 @@ virDomainSEVDefParseXML(xmlNodePtr sevNode,
         virReportError(VIR_ERR_XML_ERROR, "%s",
                        _("Invalid format for launch security "
                          "reduced-phys-bits"));
-        goto error;
+        return NULL;
     }
 
     def->policy = policy;
@@ -14766,9 +14766,6 @@ virDomainSEVDefParseXML(xmlNodePtr sevNode,
     def->session = virXPathString("string(./session)", ctxt);
 
     return g_steal_pointer(&def);
-
- error:
-    return NULL;
 }