change var name to prevent side effects.

[pve-storage.git] / PVE / Storage / ZFSPoolPlugin.pm

diff --git a/PVE/Storage/ZFSPoolPlugin.pm b/PVE/Storage/ZFSPoolPlugin.pm
index 5cbd1b237f03c1b96fccaf66f01ba2e5bd4060b9..b9b3bf029c5a8ed9a7dd45ae3ea6343efd132e3d 100644 (file)
--- a/PVE/Storage/ZFSPoolPlugin.pm
+++ b/PVE/Storage/ZFSPoolPlugin.pm
@@ -179,16 +179,18 @@ sub alloc_image {
     die "illegal name '$name' - sould be 'vm-$vmid-*'\n"
     if $name && $name !~ m/^vm-$vmid-/;
 
-    $name = $class->zfs_find_free_diskname($storeid, $scfg, $vmid) if !$name;
-    
-    $class->zfs_create_zvol($scfg, $name, $size);
+    my $volname = $name;
+ 
+    $volname = $class->zfs_find_free_diskname($storeid, $scfg, $vmid) if !$volname;
 
-    my $devname = "/dev/zvol/$scfg->{pool}/$name";
+    $class->zfs_create_zvol($scfg, $volname, $size);
+
+    my $devname = "/dev/zvol/$scfg->{pool}/$volname";
 
     run_command("udevadm trigger --subsystem-match block");
     system("udevadm settle --timeout 10 --exit-if-exists=${devname}");
 
-    return $name;
+    return $volname;
 }
 
 sub free_image {