On 06/12/2017 04:01 PM, Dietmar Maurer wrote:
+    # add 'unusedX' volumes to volhash
+    foreach my $key (keys %$conf) {
+       my $value = $conf->{$key};
+       if ($key =~ m/^unused/) {
+           my $drive = PVE::QemuServer::parse_drive($key, $value);

AFAIR $value is already a volid, so what is the purpose of this parse_drive
call?


In this String there is normally the size included.
So it is better to parse the string before we use it.

+           $test_volid->($drive->{file}, $drive);
+       }
+    }
+
     return $volhash;
 }

--
2.11.0


_______________________________________________
pve-devel mailing list
[email protected]
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

_______________________________________________
pve-devel mailing list
[email protected]
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to