[PATCH weston] input: fix access to invalid resource

Pekka Paalanen ppaalanen at gmail.com
Tue Nov 11 01:57:58 PST 2014


On Wed, 15 Oct 2014 10:19:33 +0200
Marek Chalupa <mchqwerty at gmail.com> wrote:

> Look OK,
> 
> it can not hurt to add this check in any case.

Yeah, I take that as Acked-by.

> On 10 October 2014 22:46, Giulio Camuffo <giuliocamuffo at gmail.com> wrote:
> 
> > the keyboard focus surface may not have a valid resource (server side
> > surface or a surface surviving its client), so check if it is valid
> > before using it.

Oh yeah, this can happen during a window closing animation, I suppose.

Pushed.


Thanks,
pq


> > ---
> >  src/input.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/input.c b/src/input.c
> > index e765221..bc58223 100644
> > --- a/src/input.c
> > +++ b/src/input.c
> > @@ -1803,7 +1803,7 @@ seat_get_keyboard(struct wl_client *client, struct
> > wl_resource *resource,
> >                                            seat->keyboard->focus_serial);
> >         }
> >
> > -       if (seat->keyboard->focus &&
> > +       if (seat->keyboard->focus && seat->keyboard->focus->resource &&
> >             wl_resource_get_client(seat->keyboard->focus->resource) ==
> > client) {
> >                 struct weston_surface *surface =
> >                         (struct weston_surface *) seat->keyboard->focus;
> > --
> > 2.1.2
> >
> > _______________________________________________
> > wayland-devel mailing list
> > wayland-devel at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
> >



More information about the wayland-devel mailing list