It only makes sense to filter for an existing udev property if the
'uid_attribute' variable is set. Otherwise multipath did not
get the uid from udev and so we might not even have udev information.
Signed-off-by: Hannes Reinecke <h...@suse.de>
---
libmultipath/configure.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/libmultipath/configure.c b/libmultipath/configure.c
index ca20ba5..0076485 100644
--- a/libmultipath/configure.c
+++ b/libmultipath/configure.c
@@ -967,7 +967,8 @@ get_refwwid (char * dev, enum devtypes dev_type, vector
pathvec, char **wwid)
return ret;
}
}
- if (pp->udev && filter_property(conf, pp->udev) > 0)
+ if (pp->udev && pp->uid_attribute &&
+ filter_property(conf, pp->udev) > 0)
return 2;
refwwid = pp->wwid;
@@ -994,7 +995,8 @@ get_refwwid (char * dev, enum devtypes dev_type, vector
pathvec, char **wwid)
return ret;
}
}
- if (pp->udev && filter_property(conf, pp->udev) > 0)
+ if (pp->udev && pp->uid_attribute &&
+ filter_property(conf, pp->udev) > 0)
return 2;
refwwid = pp->wwid;
@@ -1017,7 +1019,8 @@ get_refwwid (char * dev, enum devtypes dev_type, vector
pathvec, char **wwid)
dev);
return ret;
}
- if (pp->udev && filter_property(conf, pp->udev) > 0)
+ if (pp->udev && pp->uid_attribute &&
+ filter_property(conf, pp->udev) > 0)
return 2;
refwwid = pp->wwid;
--
2.6.6
--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
