From: John Ferlan <jferlan@redhat.com>
Date: Wed, 16 Dec 2015 16:54:04 +0000 (-0500)
Subject: storage: Fix startup issue for logical pool
X-Git-Tag: v1.3.1-rc1~164
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=8c865052b98f927fb3cc2d043e7ffff6fdcb2be9;p=thirdparty%2Flibvirt.git

storage: Fix startup issue for logical pool

Commit id '71b803ac' assumed that the storage pool source device path
was required for a 'logical' pool. This resulted in a failure to start
a pool without any device path defined.

So, adjust the virStorageBackendLogicalMatchPoolSource logic to
return success if at least the pool name matches the vgs output
when no pool source device path is/are provided.
---

diff --git a/src/storage/storage_backend_logical.c b/src/storage/storage_backend_logical.c
index c52782f9fc..f59684a837 100644
--- a/src/storage/storage_backend_logical.c
+++ b/src/storage/storage_backend_logical.c
@@ -541,6 +541,15 @@ virStorageBackendLogicalMatchPoolSource(virStoragePoolObjPtr pool)
         goto cleanup;
     }
 
+    /* If the pool has defined source device(s), then let's make sure
+     * they match as well; otherwise, matching can only occur on the
+     * pool's name.
+     */
+    if (!pool->def->source.ndevice) {
+        ret = true;
+        goto cleanup;
+    }
+
     /* Let's make sure the pool's device(s) match what the pvs output has
      * for volume group devices.
      */