libmultipath: don't treat multi-linear mappings as partitions
authorMartin Wilck <mwilck@suse.com>
Mon, 15 May 2017 15:37:18 +0000 (17:37 +0200)
committerChristophe Varoqui <christophe.varoqui@opensvc.com>
Wed, 17 May 2017 21:41:04 +0000 (23:41 +0200)
dm_type is used in libmultipath only to check whether a mapping
is "linear", with the intention to test if it represents a
"partition". This test returns TRUE also for mappings with
multiple targets, the first of which happens to be a linear
mapping into the target device. This is questionable, it's
hard to assign a "type" to such maps anyway.

Fix this by returning an error for multi-target maps.
This is analogous to the patch
"kpartx: don't treat multi-linear mappings as partitions".

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

index d73ba1a..1c6b87e 100644 (file)
@@ -566,7 +566,7 @@ out:
  * returns:
  *    1 : match
  *    0 : no match
- *   -1 : empty map
+ *   -1 : empty map, or more than 1 target
  */
 int dm_type(const char *name, char *type)
 {
@@ -588,10 +588,11 @@ int dm_type(const char *name, char *type)
                goto out;
 
        /* Fetch 1st target */
-       dm_get_next_target(dmt, NULL, &start, &length,
-                          &target_type, &params);
-
-       if (!target_type)
+       if (dm_get_next_target(dmt, NULL, &start, &length,
+                              &target_type, &params) != NULL)
+               /* multiple targets */
+               r = -1;
+       else if (!target_type)
                r = -1;
        else if (!strcmp(target_type, type))
                r = 1;
@@ -1180,9 +1181,9 @@ do_foreach_partmaps (const char * mapname,
        do {
                if (
                    /*
-                    * if devmap target is "linear"
+                    * if there is only a single "linear" target
                     */
-                   (dm_type(names->name, TGT_PART) > 0) &&
+                   (dm_type(names->name, TGT_PART) == 1) &&
 
                    /*
                     * and the uuid of the target is a partition of the