[PATCH weston 05/11] desktop-shell: Avoid NULL pointer dereference
Quentin Glidic
sardemff7+wayland at sardemff7.net
Sun Jun 19 09:16:35 UTC 2016
On 18/06/2016 19:15, Armin Krezović wrote:
> When there are no outputs present, an output pointer
> can be NULL. Dereferencing such pointer will result
> in a crash.
>
> Signed-off-by: Armin Krezović <krezovic.armin at gmail.com>
> ---
> desktop-shell/shell.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c
> index c48d41a..19a20ac 100644
> --- a/desktop-shell/shell.c
> +++ b/desktop-shell/shell.c
> @@ -2290,7 +2290,7 @@ shell_surface_set_class(struct wl_client *client,
> static void
> restore_output_mode(struct weston_output *output)
> {
> - if (output->original_mode)
> + if (output && output->original_mode)
> weston_output_mode_switch_to_native(output);
> }
>
>
Sounds good.
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>
Cheers,
--
Quentin “Sardem FF7” Glidic
More information about the wayland-devel
mailing list