[pve-devel] [PATCH FOLLOW-UP storage 3/3] rename_snapshot: fix parameter checks
Fabian Grünbichler
f.gruenbichler at proxmox.com
Thu Jul 10 17:09:52 CEST 2025
both source and target snapshot need to be provided when renaming.
Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
---
src/PVE/Storage.pm | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/PVE/Storage.pm b/src/PVE/Storage.pm
index 53965ee..b3ca094 100755
--- a/src/PVE/Storage.pm
+++ b/src/PVE/Storage.pm
@@ -2348,7 +2348,8 @@ sub rename_snapshot {
my ($cfg, $volid, $source_snap, $target_snap) = @_;
die "no volid provided\n" if !$volid;
- die "no source or target snap provided\n" if !$source_snap && !$target_snap;
+ die "no source snapshot provided\n" if !$source_snap;
+ die "no target snapshot provided\n" if !$target_snap;
my ($storeid, $volname) = parse_volume_id($volid);
--
2.39.5
More information about the pve-devel
mailing list