1 files changed, 46 insertions, 12 deletions
diff --git a/wm/bbkeys09/patches/patch-src_Netclient.cpp b/wm/bbkeys09/patches/patch-src_Netclient.cpp
index 643caaf0a43..92c71b857b0 100644
--- a/wm/bbkeys09/patches/patch-src_Netclient.cpp
+++ b/wm/bbkeys09/patches/patch-src_Netclient.cpp
@@ -1,15 +1,49 @@
-$NetBSD: patch-src_Netclient.cpp,v 1.1 2011/11/25 22:21:28 joerg Exp $
+$NetBSD: patch-src_Netclient.cpp,v 1.2 2015/11/17 15:59:13 he Exp $
 
---- src/Netclient.cpp.orig	2011-11-25 17:00:35.000000000 +0000
+Fix LP64 problem with the X11 protocol handling, from FreeBSD.
+
+--- src/Netclient.cpp.orig	2008-12-22 01:10:19.000000000 +0000
 +++ src/Netclient.cpp
-@@ -21,9 +21,9 @@
- // DEALINGS IN THE SOFTWARE.
- 
- // E_O_H_VR
-+#include <cstring>
- 
- // Methods, ideas, implementations taken from Openbox's XAtom class *sigh*
--
- #include "Netclient.h"
+@@ -141,6 +141,7 @@ bool Netclient::getValue(Window win, Ato
+   int result;
+   unsigned long maxread = nelements;
+   bool ret = False;
++  int bsize;
  
- Netclient::Netclient (const bt::Display &display)
+   // try get the first element
+   result = XGetWindowProperty(_display.XDisplay(), win, atom, 0l, 1l, False,
+@@ -149,18 +150,19 @@ bool Netclient::getValue(Window win, Ato
+   ret = (result == Success && ret_type == type && ret_size == size &&
+          nelements > 0);
+   if (ret) {
++    bsize = (size == 32) ? sizeof(long) : size/8;
+     if (ret_bytes == 0 || maxread <= nelements) {
+       // we got the whole property's value
+-      *value = new unsigned char[nelements * size/8 + 1];
+-      memcpy(*value, c_val, nelements * size/8 + 1);
++      *value = new unsigned char[nelements * bsize + 1];
++      memcpy(*value, c_val, nelements * bsize + 1);
+     } else {
+       // get the entire property since it is larger than one long
+       XFree(c_val);
+       // the number of longs that need to be retreived to get the property's
+       // entire value. The last + 1 is the first long that we retrieved above.
+       int remain = (ret_bytes - 1)/sizeof(long) + 1 + 1;
+-      if (remain > size/8 * (signed)maxread) // dont get more than the max
+-        remain = size/8 * (signed)maxread;
++      if (remain > bsize * (signed)maxread) // dont get more than the max
++        remain = bsize * (signed)maxread;
+       result = XGetWindowProperty(_display.XDisplay(), win, atom, 0l, remain, False, type,
+                                   &ret_type, &ret_size, &nelements, &ret_bytes,
+                                   &c_val);
+@@ -174,8 +176,8 @@ bool Netclient::getValue(Window win, Ato
+       if (! ret)
+         return getValue(win, atom, type, maxread, value, size);
+   
+-      *value = new unsigned char[nelements * size/8 + 1];
+-      memcpy(*value, c_val, nelements * size/8 + 1);
++      *value = new unsigned char[nelements * bsize + 1];
++      memcpy(*value, c_val, nelements * bsize + 1);
+     }    
+   }
+   if (c_val) XFree(c_val);