followup code cleanup

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>

diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm
index b94dd1225bf0970c5af0c27cb3cd163a21ddcfad..984052c908b472436898608b5985480e5d752b7a 100644 (file)
--- a/PVE/API2/Qemu.pm
+++ b/PVE/API2/Qemu.pm
@@ -854,14 +854,12 @@ __PACKAGE__->register_method({
 
        my $conf = PVE::QemuConfig->load_config($param->{vmid});
 
-       my $snapname = $param->{snapshot};
-       if ($snapname) {
+       if (my $snapname = $param->{snapshot}) {
            my $snapshot = $conf->{snapshots}->{$snapname};
-           die "snapshot '$snapname' does not exist\n"
-               if !defined($snapshot);
+           die "snapshot '$snapname' does not exist\n" if !defined($snapshot);
+
+           $snapshot->{digest} = $conf->{digest}; # keep file digest for API
 
-           # we need the digest of the file
-           $snapshot->{digest} = $conf->{digest};
            $conf = $snapshot;
        }
 

diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
index 3b642863fce715d55b66d3393fa48f480e71071b..32f52fe2b0714e94769b6a146ae9baf406a0cc05 100644 (file)
--- a/PVE/QemuServer.pm
+++ b/PVE/QemuServer.pm
@@ -5378,12 +5378,10 @@ sub vm_commandline {
 
     if ($snapname) {
        my $snapshot = $conf->{snapshots}->{$snapname};
-       die "snapshot '$snapname' does not exist\n"
-           if !defined($snapshot);
-       my $digest = $conf->{digest};
+       die "snapshot '$snapname' does not exist\n" if !defined($snapshot);
+
+       $snapshot->{digest} = $conf->{digest}; # keep file digest for API
 
-       # we need the digest of the file
-       $snapshot->{digest} = $conf->{digest};
        $conf = $snapshot;
     }