[PATCH qemu-server v3 2/6] drive: fix conversion of bps throttle limits to mbps

[Erik Fastermann <e.fastermann@proxmox.com>]

When translating the bps, bps_rd and bps_wr options to their mbps
counterparts, parse_drive stored the result of defined() instead of
the value itself. Any limit set this way thus became 0 mbps, which
QEMU treats as unlimited, so the limit was silently ignored.

The web UI only uses the mbps options, so only users setting the bps
options via the API or CLI are affected.

Signed-off-by: Erik Fastermann <e.fastermann@proxmox.com>
---
 src/PVE/QemuServer/Drive.pm | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/PVE/QemuServer/Drive.pm b/src/PVE/QemuServer/Drive.pm
index 0b573717..bccf0371 100644
--- a/src/PVE/QemuServer/Drive.pm
+++ b/src/PVE/QemuServer/Drive.pm
@@ -867,7 +867,7 @@ sub parse_drive {
 
     my $error = 0;
     foreach my $opt (qw(bps bps_rd bps_wr)) {
-        if (my $bps = defined(delete $res->{$opt})) {
+        if (defined(my $bps = delete $res->{$opt})) {
             if (defined($res->{"m$opt"})) {
                 warn "both $opt and m$opt specified\n";
                 ++$error;
-- 
2.47.3