diff options
author | Giovanni Campagna <gcampagn@redhat.com> | 2013-07-26 15:31:17 +0200 |
---|---|---|
committer | Giovanni Campagna <gcampagna@src.gnome.org> | 2013-08-18 00:47:53 +0200 |
commit | cd20f1bc0b4cc9c120e2916bdfd3ed32a5371b06 (patch) | |
tree | 1eeb3642558403202e7955fbf7c0c91efeeae20b | |
parent | 8b52782ed4ce07a52c0f681f43a01458780896ff (diff) | |
download | mutter-cd20f1bc0b4cc9c120e2916bdfd3ed32a5371b06.tar.gz |
MonitorManager: ignore configuration changes that disable all outputs
If we compute a screen size of 0 in either direction, we crash
later on, as it is invalid for clutter, cogl and X.
https://bugzilla.gnome.org/show_bug.cgi?id=705670
-rw-r--r-- | src/core/monitor.c | 22 |
1 files changed, 20 insertions, 2 deletions
diff --git a/src/core/monitor.c b/src/core/monitor.c index b76ed9ff9..aaa67c78a 100644 --- a/src/core/monitor.c +++ b/src/core/monitor.c @@ -1580,6 +1580,7 @@ meta_monitor_manager_handle_apply_configuration (MetaDBusDisplayConfig *skeleto GVariant *properties; guint crtc_id; int new_mode, x, y; + int new_screen_width, new_screen_height; guint transform; guint output_id; GPtrArray *crtc_infos, *output_infos; @@ -1598,6 +1599,7 @@ meta_monitor_manager_handle_apply_configuration (MetaDBusDisplayConfig *skeleto (GDestroyNotify) meta_output_info_free); /* Validate all arguments */ + new_screen_width = 0; new_screen_height = 0; g_variant_iter_init (&crtc_iter, crtcs); while (g_variant_iter_loop (&crtc_iter, "(uiiiuaua{sv})", &crtc_id, &new_mode, &x, &y, &transform, @@ -1657,9 +1659,17 @@ meta_monitor_manager_handle_apply_configuration (MetaDBusDisplayConfig *skeleto "Invalid CRTC geometry"); return TRUE; } + + new_screen_width = MAX (new_screen_width, x + width); + new_screen_height = MAX (new_screen_height, y + height); + crtc_info->x = x; + crtc_info->y = y; + } + else + { + crtc_info->x = 0; + crtc_info->y = 0; } - crtc_info->x = x; - crtc_info->y = y; if (transform < WL_OUTPUT_TRANSFORM_NORMAL || transform > WL_OUTPUT_TRANSFORM_FLIPPED_270 || @@ -1720,6 +1730,14 @@ meta_monitor_manager_handle_apply_configuration (MetaDBusDisplayConfig *skeleto g_ptr_array_add (crtc_infos, crtc_info); } + if (new_screen_width == 0 || new_screen_height == 0) + { + g_dbus_method_invocation_return_error (invocation, G_DBUS_ERROR, + G_DBUS_ERROR_INVALID_ARGS, + "Refusing to disable all outputs"); + return TRUE; + } + g_variant_iter_init (&output_iter, outputs); while (g_variant_iter_loop (&output_iter, "(u@a{sv})", &output_id, &properties)) { |