From 40ee921fffd37febddabe263c9e5e88320c52841 Mon Sep 17 00:00:00 2001 From: Axel Davy Date: Fri, 8 Nov 2013 12:59:34 +0100 Subject: [PATCH] Do not set output->current_mode in compositor.c The field is already set - correctly - in the backend switch_mode. setting output->current_mode to mode in compositor.c leads to bugs, since mode can be freed by the shell. For example, the shell allocates it on the stack for WL_SHELL_SURFACE_FULLSCREEN_METHOD_DRIVER Signed-off-by: Axel Davy --- src/compositor.c | 1 - 1 file changed, 1 deletion(-) diff --git a/src/compositor.c b/src/compositor.c index cfcb273f..4668c2aa 100644 --- a/src/compositor.c +++ b/src/compositor.c @@ -137,7 +137,6 @@ weston_output_switch_mode(struct weston_output *output, struct weston_mode *mode if (ret < 0) return ret; - output->current_mode = mode; output->current_scale = scale; break; case WESTON_MODE_SWITCH_RESTORE_NATIVE: