[PATCH] dbus-launch: use InputOnly X window

Eamon Walsh ewalsh at tycho.nsa.gov
Thu Mar 19 21:26:42 PDT 2009


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 at tycho.nsa.gov>
---

diff --git a/tools/dbus-launch-x11.c b/tools/dbus-launch-x11.c
index 442e9ba..b182418 100644
--- a/tools/dbus-launch-x11.c
+++ b/tools/dbus-launch-x11.c
@@ -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 */




-- 
Eamon Walsh <ewalsh at tycho.nsa.gov>
National Security Agency



More information about the dbus mailing list