Re: Window Maker

[Pierre Pronchery <khorben%defora.org@localhost>]

                Hey,

Andras Barna wrote:
> Lorenz, can you send the patch which you talked about in pr #37054 ?

Here is mine.

HTH,
-- 
khorben

$NetBSD$

--- src/wmspec.c.orig   2007-10-02 15:40:36.000000000 +0200
+++ src/wmspec.c
@@ -27,6 +27,8 @@
  * This file needs to be checked for all calls to XGetWindowProperty() and
  * proper checks need to be made on the returned values. Only checking for
  * return to be Success is not enough. -Dan
+ * This file also needs to be checked for all calls to XChangeProperty(),
+ * where the fifth argument needs to be aligned on the size of long.
  */
 
 #include "wconfig.h"
@@ -392,7 +394,8 @@ wNETWMGetCurrentDesktopFromHint(WScreen 
 static CARD32*
 findBestIcon(CARD32 *data, unsigned long items)
 {
-    int size, wanted, d, distance;
+    int size, wanted;
+    long d, distance;
     unsigned long i;
     CARD32 *icon;
 
@@ -487,7 +490,7 @@ updateIconImage(WScreen *scr, WWindow *w
 static void
 updateShowDesktop(WScreen * scr, Bool show)
 {
-    CARD32 foo;
+    long foo;
 
     foo = (show == True);
     XChangeProperty(dpy, scr->root_win, net_showing_desktop, XA_CARDINAL, 32,
@@ -793,7 +796,7 @@ updateClientListStacking(WScreen *scr, W
 static void
 updateWorkspaceCount(WScreen *scr) /* changeable */
 {
-    CARD32 count;
+    long count;
 
     count = scr->workspace_count;
 
@@ -805,7 +808,7 @@ updateWorkspaceCount(WScreen *scr) /* ch
 static void
 updateCurrentWorkspace(WScreen *scr) /* changeable */
 {
-    CARD32 count;
+    long count;
 
     count = scr->current_workspace;
 
@@ -852,7 +855,7 @@ updateFocusHint(WScreen *scr, WWindow *w
 static void
 updateWorkspaceHint(WWindow *wwin, Bool fake, Bool del)
 {
-    CARD32 l;
+    long l;
 
     if (del) {
         XDeleteProperty(dpy, wwin->client_win, net_wm_desktop);