virStorageSourceGetMetadata: Refactor cleanup

Use g_autoptr for the hash table and remove the 'ret' variable.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>

diff --git a/src/storage_file/storage_source.c b/src/storage_file/storage_source.c
index 9d3761f5bd3c83dfcbbc41b1ecac5c20151ea281..51f97b13ef326e92983b98e5be50bcab7da1ddad 100644 (file)
--- a/src/storage_file/storage_source.c
+++ b/src/storage_file/storage_source.c
@@ -1421,17 +1421,13 @@ virStorageSourceGetMetadata(virStorageSourcePtr src,
                             uid_t uid, gid_t gid,
                             bool report_broken)
 {
-    GHashTable *cycle = NULL;
+    g_autoptr(GHashTable) cycle = virHashNew(NULL);
     virStorageType actualType = virStorageSourceGetActualType(src);
-    int ret = -1;
 
     VIR_DEBUG("path=%s format=%d uid=%u gid=%u report_broken=%d",
               src->path, src->format, (unsigned int)uid, (unsigned int)gid,
               report_broken);
 
-    if (!(cycle = virHashNew(NULL)))
-        return -1;
-
     if (src->format <= VIR_STORAGE_FILE_NONE) {
         if (actualType == VIR_STORAGE_TYPE_DIR)
             src->format = VIR_STORAGE_FILE_DIR;
@@ -1439,9 +1435,6 @@ virStorageSourceGetMetadata(virStorageSourcePtr src,
             src->format = VIR_STORAGE_FILE_RAW;
     }
 
-    ret = virStorageSourceGetMetadataRecurse(src, src, uid, gid,
-                                             report_broken, cycle, 1);
-
-    virHashFree(cycle);
-    return ret;
+    return virStorageSourceGetMetadataRecurse(src, src, uid, gid,
+                                              report_broken, cycle, 1);
 }