[PATCH 02/18] util: storagefile: Always store raw backing name in the metadata

[Peter Krempa <pkrempa@xxxxxxxxxx>]

Don't use the backingStoreRaw as a indication of broken chains. Fill it
always and tweak the broken image chain detector to avoid changing the
semantics.

The new semantics to detect a broken chain is the presence of string in
backingStoreRaw but the lack of the backing chain metadata structure in
the chain.

Now that the raw backing store name is always filled there's no need to
pass the raw name variable separately to fill in case the backing is not
a file. Tweak the function so that it can handle a NULL in that case.
---
 src/util/virstoragefile.c | 51 +++++++++++++++++++----------------------------
 1 file changed, 21 insertions(+), 30 deletions(-)

diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c
index 73cfef0..5fbb6e7 100644
--- a/src/util/virstoragefile.c
+++ b/src/util/virstoragefile.c
@@ -864,8 +864,7 @@ virStorageFileGetMetadataInternal(const char *path,
     }

     if (fileTypeInfo[format].getBackingStore != NULL) {
-        char *backing = NULL;
-        int store = fileTypeInfo[format].getBackingStore(&backing,
+        int store = fileTypeInfo[format].getBackingStore(&meta->backingStoreRaw,
                                                          backingFormat,
                                                          buf, len);
         if (store == BACKING_STORE_INVALID)
@@ -874,15 +873,10 @@ virStorageFileGetMetadataInternal(const char *path,
         if (store == BACKING_STORE_ERROR)
             goto cleanup;

-        if (backing != NULL) {
-            if (VIR_STRDUP(meta->backingStore, backing) < 0) {
-                VIR_FREE(backing);
-                goto cleanup;
-            }
-            if (virStorageIsFile(backing)) {
-                meta->backingStoreRaw = meta->backingStore;
-                meta->backingStore = NULL;
-                if (virFindBackingFile(directory, backing,
+        if (meta->backingStoreRaw) {
+            if (virStorageIsFile(meta->backingStoreRaw)) {
+                if (virFindBackingFile(directory,
+                                       meta->backingStoreRaw,
                                        backingDirectory,
                                        &meta->backingStore) < 0) {
                     /* the backing file is (currently) unavailable, treat this
@@ -894,11 +888,15 @@ virStorageFileGetMetadataInternal(const char *path,

                 }
             } else {
-                *backingStore = backing;
-                backing = NULL;
+                if (VIR_STRDUP(meta->backingStore, meta->backingStoreRaw) < 0)
+                    goto cleanup;
+
+                if (backingStore &&
+                    VIR_STRDUP(*backingStore, meta->backingStoreRaw) < 0)
+                    goto cleanup;
+
                 *backingFormat = VIR_STORAGE_FILE_RAW;
             }
-            VIR_FREE(backing);
         } else {
             meta->backingStore = NULL;
             *backingFormat = VIR_STORAGE_FILE_NONE;
@@ -1085,7 +1083,7 @@ virStorageFileGetMetadataFromFDInternal(const char *path,

     ret = virStorageFileGetMetadataInternal(path, canonPath, directory,
                                             buf, len, format, meta,
-                                            &meta->backingStoreRaw,
+                                            NULL,
                                             backingFormat, backingDirectory);

     if (ret == 0) {
@@ -1296,31 +1294,24 @@ virStorageFileChainGetBroken(virStorageFileMetadataPtr chain,
                              char **brokenFile)
 {
     virStorageFileMetadataPtr tmp;
-    int ret = -1;

     *brokenFile = NULL;

     if (!chain)
         return 0;

-    tmp = chain;
-    while (tmp) {
+    for (tmp = chain; tmp; tmp = tmp->backingMeta) {
         /* Break when we hit end of chain; report error if we detected
          * a missing backing file, infinite loop, or other error */
-       if (!tmp->backingStoreRaw)
-           break;
-       if (!tmp->backingStore) {
-           if (VIR_STRDUP(*brokenFile, tmp->backingStoreRaw) < 0)
-               goto error;
-           break;
-       }
-       tmp = tmp->backingMeta;
-    }
+        if (!tmp->backingMeta && tmp->backingStoreRaw) {
+            if (VIR_STRDUP(*brokenFile, tmp->backingStoreRaw) < 0)
+                return -1;

-    ret = 0;
+           return 0;
+        }
+    }

- error:
-    return ret;
+    return 0;
 }


-- 
1.9.1

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list