On Sun, 19 Jun 2016 11:16:35 +0200 Quentin Glidic <[email protected]> wrote:
> 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ć <[email protected]> > > --- > > 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 <[email protected]> Reviewed-by: Pekka Paalanen <[email protected]> fullscreen-shell.c has nother place like this. Thanks, pq
pgp67aqe_Kx1v.pgp
Description: OpenPGP digital signature
_______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
