[dm-devel] [PATCH v2 2/3] libmultipath: cleanup uid_attribute checking code
Benjamin Marzinski
bmarzins at redhat.com
Wed Feb 24 06:33:21 UTC 2021
In get_uid(), if pp->getuid is NULL, multipath will check the
pp->uid_attribute to get the wwid. If pp->uid_attribute is NULL,
nothing will happen in that block of code, because both udev_available
and has_uid_fallback() are false if pp->uid_attribute is NULL. So
instead of multiple checks if pp->uid_attribute is NULL, just check once
for the code block.
Signed-off-by: Benjamin Marzinski <bmarzins at redhat.com>
---
libmultipath/discovery.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)
diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
index 3a06f319..40727fa3 100644
--- a/libmultipath/discovery.c
+++ b/libmultipath/discovery.c
@@ -2183,22 +2183,21 @@ get_uid (struct path * pp, int path_state, struct udev_device *udev,
} else
len = strlen(pp->wwid);
origin = "callout";
- } else {
- bool valid_uid_attr = pp->uid_attribute
- && *pp->uid_attribute;
- bool empty_uid_attr = pp->uid_attribute
- && !*pp->uid_attribute;
- bool udev_available = udev && valid_uid_attr;
+ } else if (pp->uid_attribute) {
+ /* if the uid_attribute is an empty string skip udev checking */
+ bool check_uid_attr = udev && *pp->uid_attribute;
- if (udev_available) {
+ if (check_uid_attr) {
len = get_udev_uid(pp, pp->uid_attribute, udev);
origin = "udev";
if (len == 0)
condlog(1, "%s: empty udev uid", pp->dev);
}
- if ((!udev_available || (len <= 0 && allow_fallback))
+ if ((!check_uid_attr || (len <= 0 && allow_fallback))
&& has_uid_fallback(pp)) {
- if (!udev || !empty_uid_attr)
+ /* if udev wasn't set or we failed in get_udev_uid()
+ * log at a higher priority */
+ if (!udev || check_uid_attr)
used_fallback = 1;
len = uid_fallback(pp, path_state, &origin);
}
--
2.17.2
More information about the dm-devel
mailing list