From d81508fbb9d69dfaa621d292e1e435e3e55bf84c Mon Sep 17 00:00:00 2001
From: Dominik Csapak <d.csapak@proxmox.com>
Date: Fri, 19 Feb 2016 10:43:57 +0100
Subject: [PATCH] deactivate volumes if vm start command fails

prevents volumes to be active when they are not actually in use
this is a cleanup of Dhaussy Alexandre's patch from 02/15/2016

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
---
 PVE/QemuServer.pm | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
index 3d3ef6db..d3f4da2e 100644
--- a/PVE/QemuServer.pm
+++ b/PVE/QemuServer.pm
@@ -4174,8 +4174,12 @@ sub vm_start {
 
 	eval  { run_command($cmd, timeout => $statefile ? undef : 30,
 		    umask => 0077); };
-	my $err = $@;
-	die "start failed: $err" if $err;
+
+	if (my $err = $@) {
+	    # deactivate volumes if start fails
+	    eval { PVE::Storage::deactivate_volumes($storecfg, $vollist); };
+	    die "start failed: $err" if $err;
+	}
 
 	print "migration listens on $migrate_uri\n" if $migrate_uri;