Thanks Guns that worked!<br><br>Oh boy I let open a can of worms, LOL...<br><br>So we have a Feh bug do we? If so PLEASE report this also to his Github with patch solution! :)<br><br>So the last patch Brian put up; feh-mwmhints2.patch, this is the correct solution?<br>
<br>Thanks...<br><br><div class="gmail_quote">On Sun, Jul 28, 2013 at 10:23 AM, Jim Rees <span dir="ltr"><<a href="mailto:rees@umich.edu" target="_blank">rees@umich.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">Brian Mattern wrote:<br>
<br>
  On Sun, 28 Jul 2013, Jim Rees wrote:<br>
<br>
  > Brian Mattern wrote:<br>
  ><br>
  >   The correct fix is attached. On 64-bit machines, X apparently means<br>
  >   "long" when it says "32", and NOT 32 bits. So, it actually expects 40<br>
  >   bytes of data to be passed.<br>
  ><br>
  > That doesn't make sense. Openbox has its own definition of mwmhints, and the<br>
  > elements are declared as guint, which is an unsigned int. I wonder if<br>
  > openbox is doing the right thing.<br>
<br>
  The manpage for XChangeProperty states:<br>
<br>
    If the specified format is 8, the property data must be a char array.<br>
    If the specified format is 16, the property data must be a short array.<br>
    If the specified format is 32, the property data must be a long array.<br>
<br>
  It may be unintuitive and perhaps a bit silly, but it is what it is.<br>
<br>
  You can also look at get_all() in openbox/obt/prop.c, which casts the<br>
  data to gulong and then copies the value into a guint32.<br>
  (get_all() gets called by OBT_PROP_GETA32() in client_get_mwm_hints() in<br>
  openbox/openbox/client.c)<br>
<br>
</div>Ah, that makes sense. A bit silly as you say. Thanks.<br>
<div class="im"><br>
  ><br>
  > Also I'd be much happier if you left the memset in.<br>
<br>
  The code is correct either way, so I'm ambivalent. But, I'll add it in<br>
  for good form.<br>
<br>
</div>The issue is that it leaks info from whatever is on the stack. Probably not<br>
a big deal in this case, but the kind of thing that leads to security<br>
problems.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
openbox mailing list<br>
<a href="mailto:openbox@icculus.org">openbox@icculus.org</a><br>
<a href="http://icculus.org/mailman/listinfo/openbox" target="_blank">http://icculus.org/mailman/listinfo/openbox</a><br>
</div></div></blockquote></div><br>