dbus-launch: use InputOnly X window
authorEamon Walsh <ewalsh@tycho.nsa.gov>
Fri, 20 Mar 2009 04:26:42 +0000 (00:26 -0400)
committerColin Walters <walters@verbum.org>
Wed, 6 May 2009 16:51:17 +0000 (12:51 -0400)
Working on SELinux policy for X, and came across this issue in dbus-launch:

Windows created for use as property/selection placeholders should be of
class InputOnly, since no drawing is ever done to them.

Signed-off-by: Eamon Walsh <ewalsh@tycho.nsa.gov>
Signed-off-by: Thiago Macieira <thiago@kde.org>
tools/dbus-launch-x11.c

index 442e9ba..b182418 100644 (file)
@@ -361,9 +361,9 @@ set_address_in_x11(char *address, pid_t pid)
     }
 
   /* Create our window */
-  wid = XCreateSimpleWindow (xdisplay, RootWindow (xdisplay, 0), -20, -20, 10, 10,
-                             0, WhitePixel (xdisplay, 0),
-                             BlackPixel (xdisplay, 0));
+  wid = XCreateWindow (xdisplay, RootWindow (xdisplay, 0), -20, -20, 10, 10,
+                       0, CopyFromParent, InputOnly, CopyFromParent,
+                       0, NULL);
   verbose ("Created window %d\n", wid);
 
   /* Save the property in the window */