Signed-off-by: Ján Tomko <jtomko@xxxxxxxxxx> --- tests/vmx2xmltest.c | 19 +++++++------------ tests/xml2vmxtest.c | 15 +++++---------- 2 files changed, 12 insertions(+), 22 deletions(-) diff --git a/tests/vmx2xmltest.c b/tests/vmx2xmltest.c index b7f232eb77..0974168cb4 100644 --- a/tests/vmx2xmltest.c +++ b/tests/vmx2xmltest.c @@ -68,7 +68,6 @@ testCapsInit(void) static int testCompareFiles(const char *vmx, const char *xml, bool should_fail_parse) { - int ret = -1; g_autofree char *vmxData = NULL; g_autofree char *formatted = NULL; g_autoptr(virDomainDef) def = NULL; @@ -79,31 +78,27 @@ testCompareFiles(const char *vmx, const char *xml, bool should_fail_parse) def = virVMXParseConfig(&ctx, xmlopt, caps, vmxData); if (should_fail_parse) { if (!def) - ret = 0; + return 0; else VIR_TEST_DEBUG("passed instead of expected failure"); - goto cleanup; + return -1; } if (!def) - goto cleanup; + return -1; if (!virDomainDefCheckABIStability(def, def, xmlopt)) { fprintf(stderr, "ABI stability check failed on %s", vmx); - goto cleanup; + return -1; } if (!(formatted = virDomainDefFormat(def, xmlopt, VIR_DOMAIN_DEF_FORMAT_SECURE))) - goto cleanup; + return -1; if (virTestCompareToFile(formatted, xml) < 0) - goto cleanup; + return -1; - ret = 0; - - cleanup: - - return ret; + return 0; } struct testInfo { diff --git a/tests/xml2vmxtest.c b/tests/xml2vmxtest.c index 8a03237b16..60185746a8 100644 --- a/tests/xml2vmxtest.c +++ b/tests/xml2vmxtest.c @@ -70,7 +70,6 @@ testCapsInit(void) static int testCompareFiles(const char *xml, const char *vmx, int virtualHW_version) { - int result = -1; g_autofree char *formatted = NULL; g_autoptr(virDomainDef) def = NULL; @@ -78,25 +77,21 @@ testCompareFiles(const char *xml, const char *vmx, int virtualHW_version) VIR_DOMAIN_DEF_PARSE_INACTIVE); if (def == NULL) - goto failure; + return -1; if (!virDomainDefCheckABIStability(def, def, xmlopt)) { fprintf(stderr, "ABI stability check failed on %s", xml); - goto failure; + return -1; } formatted = virVMXFormatConfig(&ctx, xmlopt, def, virtualHW_version); if (formatted == NULL) - goto failure; + return -1; if (virTestCompareToFile(formatted, vmx) < 0) - goto failure; + return -1; - result = 0; - - failure: - - return result; + return 0; } struct testInfo { -- 2.31.1