[libvirt PATCH v2 12/12] tests: nodedev: remove unnecessary cleanup label

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Now that the last cleanup task was removed in the previous commit, just
remove the label and return early on error rather than goto cleanup.

Signed-off-by: Jonathon Jongsma <jjongsma@xxxxxxxxxx>
---
 tests/nodedevmdevctltest.c | 20 ++++++++------------
 1 file changed, 8 insertions(+), 12 deletions(-)

diff --git a/tests/nodedevmdevctltest.c b/tests/nodedevmdevctltest.c
index 65286761c1..8ba1d2da70 100644
--- a/tests/nodedevmdevctltest.c
+++ b/tests/nodedevmdevctltest.c
@@ -44,7 +44,6 @@ testMdevctlCmd(virMdevctlCommand cmd_type,
     g_autoptr(virNodeDeviceDef) def = NULL;
     g_auto(virBuffer) buf = VIR_BUFFER_INITIALIZER;
     const char *actualCmdline = NULL;
-    int ret = -1;
     g_autofree char *outbuf = NULL;
     g_autofree char *errbuf = NULL;
     g_autofree char *stdinbuf = NULL;
@@ -64,18 +63,18 @@ testMdevctlCmd(virMdevctlCommand cmd_type,
             break;
         case MDEVCTL_CMD_LAST:
         default:
-            goto cleanup;
+            return -1;
     }
 
     if (!(def = virNodeDeviceDefParseFile(mdevxml, create, VIRT_TYPE)))
-        goto cleanup;
+        return -1;
 
     /* this function will set a stdin buffer containing the json configuration
      * of the device. The json value is captured in the callback above */
     cmd = nodeDeviceGetMdevctlCommand(def, cmd_type, &outbuf, &errbuf);
 
     if (!cmd)
-        goto cleanup;
+        return -1;
 
     if (create)
         virCommandSetDryRun(dryRunToken, &buf, true, true,
@@ -84,21 +83,18 @@ testMdevctlCmd(virMdevctlCommand cmd_type,
         virCommandSetDryRun(dryRunToken, &buf, true, true, NULL, NULL);
 
     if (virCommandRun(cmd, NULL) < 0)
-        goto cleanup;
+        return -1;
 
     if (!(actualCmdline = virBufferCurrentContent(&buf)))
-        goto cleanup;
+        return -1;
 
     if (virTestCompareToFileFull(actualCmdline, cmdfile, false) < 0)
-        goto cleanup;
+        return -1;
 
     if (create && virTestCompareToFile(stdinbuf, jsonfile) < 0)
-        goto cleanup;
-
-    ret = 0;
+        return -1;
 
- cleanup:
-    return ret;
+    return 0;
 }
 
 
-- 
2.26.3




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux