libmultipath: add dm_get_multipath
authorMartin Wilck <mwilck@suse.com>
Sat, 2 Sep 2017 22:38:43 +0000 (00:38 +0200)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Wed, 20 Sep 2017 15:48:07 +0000 (17:48 +0200)
This function is simply factored out from dm_get_maps.
No functional difference.

Signed-off-by: Martin Wilck <mwilck@suse.com>
libmultipath/devmapper.c
libmultipath/devmapper.h

index 6e3abe3..fcac6bc 100644 (file)
@@ -1004,6 +1004,31 @@ dm_disablegroup(char * mapname, int index)
        return dm_groupmsg("disable", mapname, index);
 }
 
+struct multipath *dm_get_multipath(const char *name)
+{
+       struct multipath *mpp = NULL;
+
+       mpp = alloc_multipath();
+       if (!mpp)
+               return NULL;
+
+       mpp->alias = STRDUP(name);
+
+       if (!mpp->alias)
+               goto out;
+
+       if (dm_get_map(name, &mpp->size, NULL))
+               goto out;
+
+       dm_get_uuid(name, mpp->wwid);
+       dm_get_info(name, &mpp->dmi);
+
+       return mpp;
+out:
+       free_multipath(mpp, KEEP_PATHS);
+       return NULL;
+}
+
 int
 dm_get_maps (vector mp)
 {
@@ -1036,24 +1061,12 @@ dm_get_maps (vector mp)
                if (!dm_is_mpath(names->name))
                        goto next;
 
-               mpp = alloc_multipath();
-
+               mpp = dm_get_multipath(names->name);
                if (!mpp)
                        goto out;
 
-               mpp->alias = STRDUP(names->name);
-
-               if (!mpp->alias)
-                       goto out1;
-
-               if (dm_get_map(names->name, &mpp->size, NULL))
-                       goto out1;
-
-               dm_get_uuid(names->name, mpp->wwid);
-               dm_get_info(names->name, &mpp->dmi);
-
                if (!vector_alloc_slot(mp))
-                       goto out1;
+                       goto out;
 
                vector_set_slot(mp, mpp);
                mpp = NULL;
@@ -1064,8 +1077,6 @@ next:
 
        r = 0;
        goto out;
-out1:
-       free_multipath(mpp, KEEP_PATHS);
 out:
        dm_task_destroy (dmt);
        return r;
index 5c824a2..62e14d1 100644 (file)
@@ -65,6 +65,7 @@ int dm_rename (const char * old, char * new, char * delim, int skip_kpartx);
 int dm_reassign(const char * mapname);
 int dm_reassign_table(const char *name, char *old, char *new);
 int dm_setgeometry(struct multipath *mpp);
+struct multipath *dm_get_multipath(const char *name);
 
 #define VERSION_GE(v, minv) ( \
        (v[0] > minv[0]) || \