ok thanks...<br><br>Does anyone know what command(s) I can use in the menu.xml to run Feh to set wallpaper, so I can hit enter on the keyboard to set it and also right click on the image to bring up the Feh menu?<br><br>This command allows you to hit enter on the keyboard to set a background image, but there is no menu appering when doing a right click on the image;<br>
<br>feh -Tbgpreview "/home/foo/Wallpaper/"<br><br>This command gives a menu when right click on the image, but you can't press enter to set it;<br><br>feh --zoom fill -g 800x600 "/home/foo/Wallpaper/"<br>
<div class="gmail_quote"><br><br>Thanks...<br><br><br>On Sun, Jul 28, 2013 at 6:03 PM, Brian Mattern <span dir="ltr"><<a href="mailto:rephorm@rephorm.com" target="_blank">rephorm@rephorm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The attached patch is the same as the prior, but also includes<br>
initialization of the mwmhints struct, so that no random bits get passed<br>
on to the xserver.<br>
<br>
This is the current state of the pull request on github to fix this bug.<br>
<span class="HOEnZb"><font color="#888888"><br>
Brian<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
On Sun, 28 Jul 2013, Das wrote:<br>
<br>
> 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<br>
> with patch solution! :)<br>
><br>
> So the last patch Brian put up; feh-mwmhints2.patch, this is the correct<br>
> solution?<br>
><br>
> Thanks...<br>
><br>
> On Sun, Jul 28, 2013 at 10:23 AM, Jim Rees <<a href="mailto:rees@umich.edu">rees@umich.edu</a>> wrote:<br>
><br>
> > 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,<br>
> > 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>
> > Ah, that makes sense. A bit silly as you say. Thanks.<br>
> ><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>
> > 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>
> > _______________________________________________<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>
> ><br>
<br>
> _______________________________________________<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>
<br>
</div></div><br>_______________________________________________<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>
<br></blockquote></div><br>