[PATCH 8/9] qemuProcessStartValidateGraphics: Remove redundant checks for RDP protocol features

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Peter Krempa <pkrempa@xxxxxxxxxx>

Both the 'replaceUser' and 'multiUser' field are already validated in
qemuValidateDomainDeviceDefRDPGraphics.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
 src/qemu/qemu_process.c | 10 ----------
 1 file changed, 10 deletions(-)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index b99602c5ed..920adef7df 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -5540,16 +5540,6 @@ qemuProcessStartValidateGraphics(virDomainObj *vm)
             break;

         case VIR_DOMAIN_GRAPHICS_TYPE_RDP:
-            if (graphics->data.rdp.multiUser) {
-                virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                               _("qemu-rdp doesn't support the 'multiUser' attribute."));
-                return -1;
-            }
-            if (graphics->data.rdp.replaceUser) {
-                virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
-                               _("qemu-rdp doesn't support the 'replaceUser' attribute."));
-                return -1;
-            }
             if (!virDomainDefHasDBus(vm->def, false)) {
                 virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                                _("qemu-rdp support requires a D-Bus bus graphics device."));
-- 
2.49.0




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux