[Bug 789229] Re: using a blacklist and a blacklist_exception section results in multipathd filtering out paths it shouldn't

[Serge Hallyn]

Hi Craig,

actually, I as about to upload this, but when checking the git commit,
it also fixes up main.c:

diff --git a/multipathd/main.c b/multipathd/main.c
index f887ad0..e65834a 100644
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -368,7 +368,7 @@ ev_add_path (char * devname, struct vectors * vecs)
                condlog(0, "%s: failed to get path uid", devname);
                return 1; /* leave path added to pathvec */
        }
-       if (filter_path(conf, pp)){
+       if (filter_path(conf, pp) > 0){
                int i = find_slot(vecs->pathvec, (void *)pp);
                if (i != -1)
                        vector_del_slot(vecs->pathvec, i);
@@ -1062,7 +1062,7 @@ configure (struct vectors * vecs, int start_waiters)
        path_discovery(vecs->pathvec, conf, DI_ALL);
 
        vector_foreach_slot (vecs->pathvec, pp, i){
-               if (filter_path(conf, pp)){
+               if (filter_path(conf, pp) > 0){
                        vector_del_slot(vecs->pathvec, i);
                        free_path(pp);
                        i--;

Is there a reason why you dropped that from the patch, or should it be
added to this fix?

-- 
You received this bug notification because you are a member of Ubuntu
Server Team, which is subscribed to multipath-tools in Ubuntu.
https://bugs.launchpad.net/bugs/789229

Title:
  using a blacklist and a blacklist_exception section results in
  multipathd filtering out paths it shouldn't