xinput test crashes server when touchpad clicked

Peter Hutterer peter.hutterer at who-t.net
Tue Dec 2 03:40:33 PST 2008


On Fri, Nov 28, 2008 at 08:48:03AM +0000, Magnus Kessler wrote:
> On Thursday 27 November 2008, Peter Hutterer wrote:
> > On Thu, Nov 27, 2008 at 11:49:58AM +0000, Magnus Kessler wrote:
> > > Tested-by: Magnus Kessler <Magnus.Kessler at gmx.net>
> > >
> > > That patch works fine for me. Thanks for fixing this.
> > >
> > > However, I see that the same unchecked access to p->key->xkbInfo exists
> > > in other functions in xkbEvents.c as well, notably XkbSendStateNotify
> > > and XkbSendControlsNotify (where it might be guarded by the
> > > xkb_interest field?), XkbSendMapNotify, XkbHandleBell and
> > > XkbSendActionMessage.
> > >
> > > It seems clear from the naming ("kbd") of the DeviceIntPtr parameter in
> > > those cases that above functions are intended to be called only for
> > > regular keyboard devices? Is this guaranteed?
> >
> > IIRC, these functions were always called with the VCK so that wouldn't
> > cause any problems. We can't easily do that anymore, so the "bail out if
> > it's not a keyboard" is the best approach.
> > I'll amend the patch to fix the other occurances, but it'll take me a few
> > days. If you get to it on the weekend, I'll be happy to review.
> >
> > Cheers,
> >   Peter
> 
> Looks like you beat me to it with with the patch in your latest patch-set 
> for 1.6. Please apply to main as well.

done. I got bitten by it when testing the server 1.6 changes, so I had to
spent the time fixing it. Oh well, your win :)

Cheers,
  Peter



More information about the xorg mailing list