yesBad/pve-storage
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

migrate: cleanup: $free is always defined

Browse Source
This commit is contained in:
Fabian Grünbichler
2017-05-12 15:01:08 +02:00
parent 7459cb3d91
commit 55f908a1cd
1 changed files with 2 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
PVE/Storage.pm
View File

@ -647,10 +647,8 @@ sub storage_migrate {
warn "send/receive failed, cleaning up snapshot(s)..\n" if $err; warn "send/receive failed, cleaning up snapshot(s)..\n" if $err;
eval { volume_snapshot_delete($cfg, $volid, '__migration__', 0) }; eval { volume_snapshot_delete($cfg, $volid, '__migration__', 0) };
warn "could not remove source snapshot: $@\n" if $@; warn "could not remove source snapshot: $@\n" if $@;
if (defined($free)) { eval { run_command($free) };
eval { run_command($free) }; warn "could not remove target snapshot: $@\n" if $@;
warn "could not remove target snapshot: $@\n" if $@;
}
die $err if $err; die $err if $err;
} else { } else {
die "$errstr - target type $tcfg->{type} is not valid\n"; die "$errstr - target type $tcfg->{type} is not valid\n";