From ec14a00e8c6751d0b73ca0ad5f756e4165092b87 Mon Sep 17 00:00:00 2001 From: Simon Ser Date: Sat, 26 Oct 2019 14:00:37 +0200 Subject: 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 --- sway/config/output.c | 1 + 1 file changed, 1 insertion(+) (limited to 'sway/config') diff --git a/sway/config/output.c b/sway/config/output.c index 3c36692f..50bf1155 100644 --- a/sway/config/output.c +++ b/sway/config/output.c @@ -278,6 +278,7 @@ bool apply_output_config(struct output_config *oc, struct sway_output *output) { sway_log(SWAY_ERROR, "Failed to modeset output %s", wlr_output->name); return false; } + output->current_mode = wlr_output->current_mode; if (oc && oc->scale > 0) { sway_log(SWAY_DEBUG, "Set %s scale to %f", oc->name, oc->scale); -- cgit v1.2.3