Add function: swap source and target in replication config

[pve-guest-common.git] / PVE / AbstractConfig.pm

diff --git a/PVE/AbstractConfig.pm b/PVE/AbstractConfig.pm
index fac3749bcb967c9029df221c1e55dab9f2d5c3e5..347398854e1cbb0e585d8aaa5eff2ba056e81e7c 100644 (file)
--- a/PVE/AbstractConfig.pm
+++ b/PVE/AbstractConfig.pm
@@ -480,9 +480,13 @@ sub snapshot_delete {
 
     my $prepare = 1;
 
-    my $snap;
     my $unused = [];
 
+    my $conf = $class->load_config($vmid);
+    my $snap = $conf->{snapshots}->{$snapname};
+
+    die "snapshot '$snapname' does not exist\n" if !defined($snap);
+
     $class->set_lock($vmid, 'snapshot-delete')
        if (!$drivehash); # doesn't already have a 'snapshot' lock
 
@@ -598,7 +602,7 @@ sub snapshot_rollback {
     my $repl_conf = PVE::ReplicationConfig->new();
     if ($repl_conf->check_for_existing_jobs($vmid, 1)) {
        # remove all replication snapshots
-       my $volumes = $class->get_replicatable_volumes($storecfg, $conf, 1);
+       my $volumes = $class->get_replicatable_volumes($storecfg, $vmid, $conf, 1);
        my $sorted_volids = [ sort keys %$volumes ];
 
        # remove all local replication snapshots (jobid => undef)