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

[libvirt] [PATCH] qemu: Stop recursive detection of image chains when an image is missing



Commit e0c469e58b93f852a72265919703cb6abd3779f8 that fixes the detection
of image chain wasn't complete. Iteration through the backing image
chain has to stop at the last existing image if some of the images are
missing otherwise the backing chain that is cached contains entries with
paths being set to NULL resulting to:

error: Unable to allow access for disk path (null): Bad address

Fortunately stat() is kind enough not to crash when it's presented with
a NULL argument.
---
 src/util/storage_file.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/util/storage_file.c b/src/util/storage_file.c
index 2249212..e7cbea7 100644
--- a/src/util/storage_file.c
+++ b/src/util/storage_file.c
@@ -729,6 +729,8 @@ virStorageFileGetMetadataFromBuf(int format,
                 if (meta->backingStore == NULL) {
                     /* the backing file is (currently) unavailable, treat this
                      * file as standalone */
+                    VIR_FREE(meta->backingStoreRaw);
+                    meta->backingStoreIsFile = false;
                     backingFormat = VIR_STORAGE_FILE_NONE;
                 }
             }
-- 
1.8.0


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