From 63d269d79ec51f65cf37facac52160af4c8ed80c Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Fri, 10 Jun 2016 09:26:14 +0200 Subject: [PATCH] fix unnecessary root check for unused disks we needed root@pam rights to remove an unused disk from a vm (instead of the correct Storage rights) Signed-off-by: Dominik Csapak --- PVE/API2/Qemu.pm | 1 + changelog.Debian | 2 ++ 2 files changed, 3 insertions(+) diff --git a/PVE/API2/Qemu.pm b/PVE/API2/Qemu.pm index aadfde5..c0a72c0 100644 --- a/PVE/API2/Qemu.pm +++ b/PVE/API2/Qemu.pm @@ -243,6 +243,7 @@ my $check_vm_modify_config_perm = sub { # disk checks need to be done somewhere else next if PVE::QemuServer::is_valid_drivename($opt); next if $opt eq 'cdrom'; + next if $opt =~ m/^unused\d+$/; if ($cpuoptions->{$opt} || $opt =~ m/^numa\d+$/) { $rpcenv->check_vm_perm($authuser, $vmid, $pool, ['VM.Config.CPU']); diff --git a/changelog.Debian b/changelog.Debian index d668575..a06c421 100644 --- a/changelog.Debian +++ b/changelog.Debian @@ -2,6 +2,8 @@ qemu-server (4.0-80) unstable; urgency=medium * split old style pipe open call + * fix unnecessary root check for unused disks + -- Proxmox Support Team Thu, 09 Jun 2016 18:13:03 +0200 qemu-server (4.0-79) unstable; urgency=medium -- 2.39.2