From ff00afd7fb3f72f4a023428cfe3a20584c532943 Mon Sep 17 00:00:00 2001 From: Dietmar Maurer Date: Thu, 14 Feb 2013 08:41:15 +0100 Subject: [PATCH] vmid is integer an can be 0, so we need to use defined() to check if it is set Note: The plan is to use vmid 0 to mark shared volumes. --- PVE/Storage.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/PVE/Storage.pm b/PVE/Storage.pm index c2f83a0..99aa1f0 100755 --- a/PVE/Storage.pm +++ b/PVE/Storage.pm @@ -536,7 +536,7 @@ sub vdisk_free { $plugin->parse_volname($tmpvolname); }; - if ($basename && $basevmid && $basevmid == $vmid && $basename eq $name) { + if ($basename && defined($basevmid) && $basevmid == $vmid && $basename eq $name) { die "base volume '$volname' is still in use " . "(use by '$tmpvolname')\n"; } -- 2.39.5