xserver-multidpi/miext
Jon TURNEY 70a94c5b7a rootless: fix uninitialized private key assert in non-rootless modes in Cygwin/X
IsFramedWindow() is called from miPaintWindow() if the server has
been built with ROOTLESS defined, irrespective of if RootlessInit()
has ever been called, or not.

Add a check to IsFramedWindow() to check if rootlessWindowPrivateKey
has been registered (as a proxy for checking if the rootless extension
has been initialized) so we don't go on to try to use that key,
triggering an assert.

This bug exposes what appears to be a difference in opinion about
the rootless extension between XQuartz and XWin.  XQuartz always
initializes the rootless extension, whereas XWin offers several modes
of operation, and the rootless extension is only used for one of them

That probably means that the all code under compile time guard for
ROOTLESS should be carefully checked that it doesn't also need to be
under a run-time guard

(I've reviewed the other ROOTLESS blocks in dix/events.c and
dix/window.c and they look ok -- keithp)

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Reviewed-by: Keith Packard <keithp@keithp.com>
Signed-off-by: Keith Packard <keithp@keithp.com>
2010-08-10 09:13:30 -07:00
..
cw Replace deprecated bzero with memset 2010-06-06 15:07:27 +07:00
damage Fix a couple more possible errors with input-only windows 2010-06-10 19:15:28 -07:00
rootless rootless: fix uninitialized private key assert in non-rootless modes in Cygwin/X 2010-08-10 09:13:30 -07:00
shadow Move the shadow screen private key initialization to shadowSetup 2010-06-05 21:12:41 -07:00
Makefile.am Cygwin/X: Only try to build rootless extension if multiwindow extwm mode is being built 2009-07-29 17:05:19 +01:00