Fix hang on reconfigure CLI command

Restore the vector locking outside the reconfigure() function.
Moving it inside caused a double-lock hang situation. The
first locker being uxsock_trigger(), caller of reconfigure().

Discussion on-going on wether we'd better stop locking from
uxsock_trigger().

diff --git a/multipathd/main.c b/multipathd/main.c
index cc75921..e0606b6 100644 (file)
--- a/multipathd/main.c
+++ b/multipathd/main.c
@@ -1332,7 +1332,6 @@ reconfigure (struct vectors * vecs)
        struct config * old = conf;
        int retval = 1;
 
-       lock(vecs->lock);
        /*
         * free old map and path vectors ... they use old conf state
         */
@@ -1353,7 +1352,6 @@ reconfigure (struct vectors * vecs)
                retval = 0;
        }
 
-       unlock(vecs->lock);
        return retval;
 }
 
@@ -1411,7 +1409,9 @@ sighup (int sig)
        if (running_state != DAEMON_RUNNING)
                return;
 
+       lock(gvecs->lock);
        reconfigure(gvecs);
+       unlock(gvecs->lock);
 
 #ifdef _DEBUG_
        dbg_free_final(NULL);