[Spice-devel] [PATCH spice-server] red-qxl: Make sure we have at least one monitor

Frediano Ziglio fziglio at redhat.com
Thu Sep 19 10:17:25 UTC 2019


It does not make sense to have a graphic card without a monitor.
In spice_qxl_set_max_monitors we prevent to set 0 monitors, do
the same in spice_qxl_set_device_info.

This fixes https://bugzilla.redhat.com/show_bug.cgi?id=1691721.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 server/red-qxl.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server/red-qxl.c b/server/red-qxl.c
index 03348270..dbfcd440 100644
--- a/server/red-qxl.c
+++ b/server/red-qxl.c
@@ -804,7 +804,7 @@ void spice_qxl_set_device_info(QXLInstance *instance,
     }
 
     instance->st->monitors_count = device_display_id_count;
-    instance->st->max_monitors = device_display_id_count;
+    instance->st->max_monitors = MAX(1u, device_display_id_count);
 
     reds_send_device_display_info(red_qxl_get_server(instance->st));
 }
-- 
2.21.0



More information about the Spice-devel mailing list