[libvirt] [PATCH v2 28/32] util: Rename variable in virStorageSourceNewFromBackingRelative

John Ferlan jferlan at redhat.com
Fri Feb 8 18:37:22 UTC 2019


To prepare for subsequent change to use VIR_AUTOPTR logic rename
the @ret to @def.

Signed-off-by: John Ferlan <jferlan at redhat.com>
---
 src/util/virstoragefile.c | 28 ++++++++++++++--------------
 1 file changed, 14 insertions(+), 14 deletions(-)

diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index 45399f98ce..475e797e1b 100644
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -2577,14 +2577,14 @@ static virStorageSourcePtr
 virStorageSourceNewFromBackingRelative(virStorageSourcePtr parent,
                                        const char *rel)
 {
-    virStorageSourcePtr ret;
+    virStorageSourcePtr def;
     VIR_AUTOFREE(char *) dirname = NULL;
 
-    if (VIR_ALLOC(ret) < 0)
+    if (VIR_ALLOC(def) < 0)
         return NULL;
 
     /* store relative name */
-    if (VIR_STRDUP(ret->relPath, parent->backingStoreRaw) < 0)
+    if (VIR_STRDUP(def->relPath, parent->backingStoreRaw) < 0)
         goto error;
 
     if (!(dirname = mdir_name(parent->path))) {
@@ -2593,39 +2593,39 @@ virStorageSourceNewFromBackingRelative(virStorageSourcePtr parent,
     }
 
     if (STRNEQ(dirname, "/")) {
-        if (virAsprintf(&ret->path, "%s/%s", dirname, rel) < 0)
+        if (virAsprintf(&def->path, "%s/%s", dirname, rel) < 0)
             goto error;
     } else {
-        if (virAsprintf(&ret->path, "/%s", rel) < 0)
+        if (virAsprintf(&def->path, "/%s", rel) < 0)
             goto error;
     }
 
     if (virStorageSourceGetActualType(parent) == VIR_STORAGE_TYPE_NETWORK) {
-        ret->type = VIR_STORAGE_TYPE_NETWORK;
+        def->type = VIR_STORAGE_TYPE_NETWORK;
 
         /* copy the host network part */
-        ret->protocol = parent->protocol;
+        def->protocol = parent->protocol;
         if (parent->nhosts) {
-            if (!(ret->hosts = virStorageNetHostDefCopy(parent->nhosts,
+            if (!(def->hosts = virStorageNetHostDefCopy(parent->nhosts,
                                                         parent->hosts)))
                 goto error;
 
-            ret->nhosts = parent->nhosts;
+            def->nhosts = parent->nhosts;
         }
 
-        if (VIR_STRDUP(ret->volume, parent->volume) < 0)
+        if (VIR_STRDUP(def->volume, parent->volume) < 0)
             goto error;
     } else {
         /* set the type to _FILE, the caller shall update it to the actual type */
-        ret->type = VIR_STORAGE_TYPE_FILE;
+        def->type = VIR_STORAGE_TYPE_FILE;
     }
 
  cleanup:
-    return ret;
+    return def;
 
  error:
-    virStorageSourceFree(ret);
-    ret = NULL;
+    virStorageSourceFree(def);
+    def = NULL;
     goto cleanup;
 }
 
-- 
2.20.1




More information about the libvir-list mailing list