[Spice-devel] [PATCH spice-gtk v2 1/3] display: Lower level of warning for empty monitor config

Pavel Grunt pgrunt at redhat.com
Wed Mar 16 13:46:51 UTC 2016


It is a valid case when a guest requests to turn off monitors, eg:
 xrandr --output Virtual-0 --off

Related:
https://bugzilla.redhat.com/show_bug.cgi?id=1061942

Acked-by: Frediano Ziglio <fziglio at redhat.com>
---
v2: Added more info to the commit message
---
 src/channel-display.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/channel-display.c b/src/channel-display.c
index cdaba9d..f576679 100644
--- a/src/channel-display.c
+++ b/src/channel-display.c
@@ -1813,7 +1813,11 @@ static void display_handle_monitors_config(SpiceChannel *channel, SpiceMsgIn *in
     guint i;
 
     g_return_if_fail(config != NULL);
-    g_return_if_fail(config->count > 0);
+
+    if (config->count == 0) {
+        CHANNEL_DEBUG(channel, "received empty monitor config");
+        return;
+    }
 
     CHANNEL_DEBUG(channel, "received new monitors config from guest: n: %d/%d", config->count, config->max_allowed);
 
-- 
2.7.3



More information about the Spice-devel mailing list