From: Peter Krempa <pkrempa@redhat.com>
Date: Mon, 26 May 2025 08:48:10 +0000 (+0200)
Subject: qemuProcessStartValidateGraphics: Remove redundant checks for RDP protocol features
X-Git-Tag: v11.5.0-rc1~105
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=e5eb3713437d8b374beb67d816076f4aa1f1a5d6;p=thirdparty%2Flibvirt.git

qemuProcessStartValidateGraphics: Remove redundant checks for RDP protocol features

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

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
---

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."));