storage: pass down storage type

The configuration file parser now already detects the storage type so spare the work and pass it down to the storage drivers. Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com> Signed-off-by: Adrian Reber <areber@redhat.com>

storage: pass down storage type
63c9ffa0 · Christian Brauner · 693dbdb9 · 63c9ffa0
Unverified Commit 63c9ffa0 authored Nov 28, 2017 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 15 additions and 5 deletions

storage.c src/lxc/storage/storage.c +15 -5

No files found.
--- a/src/lxc/storage/storage.c
+++ b/src/lxc/storage/storage.c
@@ -213,17 +213,27 @@ static const struct lxc_storage_type bdevs[] = {
 static const size_t numbdevs = sizeof(bdevs) / sizeof(struct lxc_storage_type);
-static const struct lxc_storage_type *get_storage_by_name(const char *name)
+static const struct lxc_storage_type *get_storage_by_name(const char *name,
+							  const char *type)
 {
+	int ret;
 	size_t i, cmplen;
-	cmplen = strcspn(name, ":");
+	if (type)
+		cmplen = strlen(type);
+	else
+		cmplen = strcspn(name, ":");
 	if (cmplen == 0)
 		return NULL;
-	for (i = 0; i < numbdevs; i++)
+	for (i = 0; i < numbdevs; i++) {
-		if (strncmp(bdevs[i].name, name, cmplen) == 0)
+		if (type)
+			ret = strncmp(bdevs[i].name, type, cmplen);
+		else
+			ret = strncmp(bdevs[i].name, name, cmplen);
+		if (ret == 0)
 			break;
+	}
 	if (i == numbdevs)
 		return NULL;
@@ -238,7 +248,7 @@ const struct lxc_storage_type *storage_query(struct lxc_conf *conf,
 	size_t i;
 	const struct lxc_storage_type *bdev;
-	bdev = get_storage_by_name(src);
+	bdev = get_storage_by_name(src, conf->rootfs.bdev_type);
 	if (bdev)
 		return bdev;