fix bug #719: vzdump - only call cleanup after prepare

8469c1ac · Dietmar Maurer · 9efa2ea7 · 8469c1ac
Commit 8469c1ac authored Oct 03, 2015 by Dietmar Maurer
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 2 deletions

VZDump.pm PVE/VZDump.pm +5 -2

No files found.
--- a/PVE/VZDump.pm
+++ b/PVE/VZDump.pm
@@ -1068,8 +1068,11 @@ sub exec_backup_task {
 	    warn $@ if $@;
 	}
-	eval { $plugin->cleanup ($task, $vmid) };
+	if (defined($task->{mode})) { 
-	warn $@ if $@;
+	    # only call cleanup when necessary (when prepare was executed)
+	    eval { $plugin->cleanup ($task, $vmid) };
+	    warn $@ if $@;
+	}
 	eval { $plugin->set_logfd (undef); };
 	warn $@ if $@;