Message ID | 1621023060-20011-7-git-send-email-bmarzins@redhat.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Delegated to: | christophe varoqui |
Headers | show |
Series | Memory issues found by coverity | expand |
On Fri, 2021-05-14 at 15:11 -0500, Benjamin Marzinski wrote: > If get_uid() is returning a different wwid in uev_update_path(), then > the uid_attribute must have already gotten updated, which was the > purpose behind calling rescan_path() in the first place. > > Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com> Reviewed-by: Martin Wilck <mwilck@suse.com> -- dm-devel mailing list dm-devel@redhat.com https://listman.redhat.com/mailman/listinfo/dm-devel
diff --git a/multipathd/main.c b/multipathd/main.c index 8e2beddd..2750f5e9 100644 --- a/multipathd/main.c +++ b/multipathd/main.c @@ -1359,7 +1359,6 @@ uev_update_path (struct uevent *uev, struct vectors * vecs) condlog(0, "%s: path wwid changed from '%s' to '%s'", uev->kernel, wwid, pp->wwid); ev_remove_path(pp, vecs, 1); - rescan_path(uev->udev); needs_reinit = 1; goto out; } else {
If get_uid() is returning a different wwid in uev_update_path(), then the uid_attribute must have already gotten updated, which was the purpose behind calling rescan_path() in the first place. Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com> --- multipathd/main.c | 1 - 1 file changed, 1 deletion(-)