multipath: check setup_multipath return value.

When setup_multipath() fails, it removes the map. So update_path_groups()
needs check the return value, and fail without touching the map anymore if
setup_multipath() fails.

Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>

diff --git a/multipathd/main.c b/multipathd/main.c
index be236cb..68ce749 100644 (file)
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -1060,7 +1060,8 @@ int update_path_groups(struct multipath *mpp, struct vectors *vecs, int refresh)
                return 1;
        }
        dm_lib_release();
-       setup_multipath(vecs, mpp);
+       if (setup_multipath(vecs, mpp) != 0)
+               return 1;
        sync_map_state(mpp);
 
        return 0;