From c9a16105d4cb4d955ed351987e894ce38be5383f Mon Sep 17 00:00:00 2001 From: Dietmar Maurer Date: Mon, 8 May 2017 07:39:47 +0200 Subject: [PATCH] PVE::ReplicationTools:sync_disk - use $lastsync instead of $job->{lastsync} Because tghis is the parsed (untainted) value, and we pass this a parameter to cleanup_snapshot. --- PVE/ReplicationTools.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/ReplicationTools.pm b/PVE/ReplicationTools.pm index b3fc965..25355dc 100644 --- a/PVE/ReplicationTools.pm +++ b/PVE/ReplicationTools.pm @@ -237,7 +237,7 @@ sub sync_guest { # delet old snapshot if exists cleanup_snapshot($disks_status, $snapname, $storage_config, $running, $ip, $lastsync) if - $job->{lastsync} != 0; + $lastsync != 0; $job->{lastsync} = $snap_time; $job->{state} = "ok";