Fix segfault in wlr_output_manager_v1_set_configuration

Calling wlr_output_manager_v1_set_configuration with an enabled output
and a NULL mode is incorrect if the output doesn't support modes.

When DPMS'ing an output, wlr_output_enable(output, false) is called.
This de-allocates the CRTC and sets wlr_output.current_mode to NULL.

Because we mark DPMS'ed outputs as enabled, we also need to provide a
correct output mode. Add a field to sway_output to hold the current
mode.

Closes: https://github.com/swaywm/wlroots/issues/1867

1 files changed, 1 insertions, 0 deletions

diff --git a/sway/desktop/output.c b/sway/desktop/output.c
index 42194d54..beb6ee79 100644
--- a/sway/desktop/output.c
+++ b/sway/desktop/output.c
@@ -662,6 +662,7 @@ static void update_output_manager_config(struct sway_server *server) {
                         root->output_layout, output->wlr_output);
                 // We mark the output enabled even if it is switched off by DPMS
                 config_head->state.enabled = output->enabled;
+                config_head->state.mode = output->current_mode;
                 if (output_box) {
                         config_head->state.x = output_box->x;
                         config_head->state.y = output_box->y;