[PATCH v2] Don't smash the event_vec if num_events differs between lib and server.

Julien Cristau jcristau at debian.org
Wed Dec 2 17:08:48 PST 2009


On Fri, Nov 27, 2009 at 10:39:50 +0100, Julien Cristau wrote:

> On Fri, Nov 27, 2009 at 13:48:38 +1000, Peter Hutterer wrote:
> 
> > If the library extension thinks there's more events to an extension than the
> > server actually has, the event_vec for the overlapping range can get
> > overwritten. This depends on the initialization order of the libraries.
> > 
> > Reported-by: Nathan Kidd
> > Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> 
> Reviewed-by: Julien Cristau <jcristau at debian.org>
> 
Sigh.  Now that I look again...  The ext_handlers[] array is shared by
all possible Displays you could pass in.  An app using more than one
will get broken data in there.

Cheers,
Julien



More information about the xorg mailing list