From 480ad9178da6e9951a0fc1aac1cac96dab10e511 Mon Sep 17 00:00:00 2001 From: "Ruben S. Montero" Date: Thu, 4 Jun 2015 20:04:10 +0200 Subject: [PATCH] feature #3782: Add missing unlocks to failure exits --- src/dm/DispatchManagerActions.cc | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/dm/DispatchManagerActions.cc b/src/dm/DispatchManagerActions.cc index ba1b61ce64..ad4a3b7bf7 100644 --- a/src/dm/DispatchManagerActions.cc +++ b/src/dm/DispatchManagerActions.cc @@ -1709,12 +1709,15 @@ int DispatchManager::disk_snapshot_revert( if (snaps == 0) { + vm->unlock(); return -1; } if (snaps->get_active_id() == snap_id) { error_str = "Snapshot is already the active one"; + + vm->unlock(); return -1; } @@ -1777,11 +1780,13 @@ int DispatchManager::disk_snapshot_delete( if (snaps == 0) { + vm->unlock(); return -1; } if (!snaps->test_delete(snap_id, error_str)) { + vm->unlock(); return -1; }