Fix wrongly paired unlocking of global lock in pvchange. (2.02.66)

diff --git a/WHATS_NEW b/WHATS_NEW
index 96a95355fc11342ecaa6b49afdb02e81a03703ea..45297f39ecfcee8236cf49bd81ca71d080fbd2e7 100644 (file)
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.80 - 
 ====================================
+  Fix wrongly paired unlocking of global lock in pvchange. (2.02.66)
   Add backtraces for backup and backup_remove fail paths.
   Detect errors from dm_task_set calls in _get_device_info (dmeventd).
   Add backtraces for archive and backup_locally in check_current_backup().

diff --git a/tools/pvchange.c b/tools/pvchange.c
index 28e71b845b484887221a7d35c84e899ed3214acb..899a8b9653af83a0adc6c509051d8d601d9633e8 100644 (file)
--- a/tools/pvchange.c
+++ b/tools/pvchange.c
@@ -288,9 +288,9 @@ int pvchange(struct cmd_context *cmd, int argc, char **argv)
                                unlock_and_free_vg(cmd, vg, sll->str);
                        }
                }
+               unlock_vg(cmd, VG_GLOBAL);
        }
 
-       unlock_vg(cmd, VG_GLOBAL);
        log_print("%d physical volume%s changed / %d physical volume%s "
                  "not changed",
                  done, done == 1 ? "" : "s",