virchrdev: Drop needless 'cleanup' label in virChrdevLockFileCreate()

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
Reviewed-by: Erik Skultety <eskultet@redhat.com>

diff --git a/src/conf/virchrdev.c b/src/conf/virchrdev.c
index 766c264472f4d396b2d7cfe8e28849471bab4d0f..800e82869e0ea5328e9ff01e67c06c538e19bdab 100644 (file)
--- a/src/conf/virchrdev.c
+++ b/src/conf/virchrdev.c
@@ -105,14 +105,13 @@ static char *virChrdevLockFilePath(const char *dev)
 static int virChrdevLockFileCreate(const char *dev)
 {
     g_autofree char *path = NULL;
-    int ret = -1;
     g_autofree char *pidStr = NULL;
     VIR_AUTOCLOSE lockfd = -1;
     pid_t pid;
 
     /* build lock file path */
     if (!(path = virChrdevLockFilePath(dev)))
-        goto cleanup;
+        return -1;
 
     /* check if a log file and process holding the lock still exists */
     if (virPidFileReadPathIfAlive(path, &pid, NULL) == 0 && pid >= 0) {
@@ -121,7 +120,7 @@ static int virChrdevLockFileCreate(const char *dev)
                        _("Requested device '%s' is locked by "
                          "lock file '%s' held by process %lld"),
                        dev, path, (long long) pid);
-        goto cleanup;
+        return -1;
     } else {
         /* clean up the stale/corrupted/nonexistent lockfile */
         unlink(path);
@@ -142,14 +141,13 @@ static int virChrdevLockFileCreate(const char *dev)
         if (errno == EACCES && geteuid() != 0) {
             VIR_DEBUG("Skipping lock file creation for device '%s in path '%s'.",
                       dev, path);
-            ret = 0;
-            goto cleanup;
+            return 0;
         }
         virReportSystemError(errno,
                              _("Couldn't create lock file for "
                                "device '%s' in path '%s'"),
                              dev, path);
-        goto cleanup;
+        return -1;
     }
 
     /* write the pid to the file */
@@ -159,15 +157,11 @@ static int virChrdevLockFileCreate(const char *dev)
                                "device '%s' in path '%s'"),
                              dev, path);
         unlink(path);
-        goto cleanup;
+        return -1;
     }
 
     /* we hold the lock */
-    ret = 0;
-
- cleanup:
-
-    return ret;
+    return 0;
 }
 
 /**