urxvt menu hangs urxvt with x input frozen

The same problem occurs with plain rxvt 1:2.6.4-14.

Same issue using urxvt on FreeBSD:

$ X -version

X.Org X Server 1.6.0
Release Date: 2009-2-25
X Protocol Version 11, Revision 0
Build Operating System: FreeBSD 7.2-PRERELEASE i386
Current Operating System: FreeBSD R-1.local. 7.2-RELEASE FreeBSD 7.2-RELEASE #0: Fri May 1 08:49:13 UTC 2009 <email address hidden>:/usr/obj/usr/src/sys/GENERIC i386
Build Date: 22 April 2009 02:40:51PM

and:

rxvt-unicode (urxvt) v9.06 - released: 2008-06-15

Any fixes?

Menu is broken for me as well.

Effects: X11 hangs except for the mouse until the
urxvt in question is killed from a console; a grab maybe?

work around is disabling popups in .Xdefaults
(use xrdb -merge ~/.Xdefaults if necessary):

(if you still want perl running)
Rxvt.perl-ext-common: defaults,-option-popup,-selection-popup
otherwise its:
Rxvt.perl-ext:
Rxvt.perl-ext-common:

likely culprits are:
/usr/lib/urxvt/urxvt.pm # contains code for popups
/usr/lib/urxvt/perl/option-popup # menu itself
/usr/lib/urxvt/perl/selection-popup # possibly innocent??

Addendum:

hang occurs _AFTER_ displaying the menu.

ubuntu jaunty 64bit, standard gnome/metacity setup
rxvt-unicode (urxvt) v9.06 - released: 2008-06-15
urxvt info toii rxvt-unicode-ml 9.06-1

urxvt >LOG 2>&1 produces NO output on hang or kill.

Info from urxvt isn't in any log.

Other possibly related messages:
maybe this one in the X log (the log itself at least has a suitable change time):
(II) AT Translated Set 2 keyboard: Device reopened after 1 attempts.
(II) SCISSORS Keyboard: Device reopened after 1 attempts.
(II) Macintosh mouse button emulation: Device reopened after 1 attempts.
(II) PS2++ Logitech Wheel Mouse: Device reopened after 1 attempts.

daemon.log (that was around the console switch to kill the urxvt, so gpm
noticed _SOMETHING_ (?)):
Jul 29 12:07:06 anuurn /usr/sbin/gpm[3306]: *** warning [daemon/old_main.c(254)]:
Jul 29 12:07:06 anuurn /usr/sbin/gpm[3306]: Data on strange file descriptor 4

upstream is also wondering:
http://lists.schmorp.de/pipermail/rxvt-unicode/2009q2.txt.gz

both popups are affected (C-middle/C-right button).

Appending straces as per request from Thomas Adam.

strace -o LOG.strace.hang-f -f urxvt -e /bin/dash
strace -o LOG.strace.hang urxvt -e /bin/dash
strace -o LOG.strace.nohang-f -f urxvt -e /bin/dash
strace -o LOG.strace.nohang urxvt -e /bin/dash

The difference being just CTRL-RIGHTBUTTON being pressed before kill or exit.
The reliability of the hangup is 100%, so getting good traces was easy :).

As per:

http://lists.schmorp.de/pipermail/rxvt-unicode/2009q3/001044.html

Please see patch attached which might help --- not tested though. Feedback welcome, if via mailing list, please.

-- Thomas Adam

Patch success confirmed.

There's still a small annoyance, as I'm required to move the mouse over another
window and back again to properly regain focus, but that's no show stopper,
and quite likely also influenced by metacity and me using sloppy focus, which
the metacity dev's hate with passion...

thanx,
Peter

Patch works for me too.

Confirming on Karmic

Attached debdiff against rxvt-unicode 9.06-1

Is this bug also present in rxvt package? I can't reproduce it (as It doesn't have a menu)

2009/8/2 Krzysztof Klimonda <email address hidden>:
> Is this bug also present in rxvt package? I can't reproduce it (as It
> doesn't have a menu)

No -- it's not present in rxvt as the code is *completely* different.
(They're written in different languages for a start.)

-- Thomas Adam

Bug isn't present in rxvt

After actually reading a linked post from ML (It was unavailable before) I've decided to use the same patch that was proposed upstream. Attaching patch and quoting description for further reference as rxvt-unicode doesn't use any patch system. I'll prepare a new debdiff shortly.

> Hi,
>
> With recent Xorg upgrades, grabbing synchronously when a popup window
> is mapped is fatal, as this sends the event loop to continually wait
> for the server to come available again which it can't do as nothing is
> ever releasing the grab. The fix then is for the callers to *not*
> force synchronous mode when calling rxvt_term::grab().
>
> See patch attached.
>
> (Note that originally I was almost tempted to just mangle the
> selection of GrabModeSync and GrabModeAsync to always be
> GrabModeAsync, however there's perfectly valid times when something
> might want to grab synchronously -- it's just nothing uses that yet.)

Here it fails on:
  ctrl+middle click
  ctrl+right click
  ctrl+alt+middle click
  ctrl+alt+right click
gonna look into trying the patch if nobody in #archlinux on freenode has already

So i apparently can not read
Sorry about that *adds to proper page*

Ok, well, despite me looking like a total retard above -- the patch *does work* on archlinux

Package FTBFS, so that will need to be fixed before anything can be uploaded:

background.C: In member function 'bool bgPixmap_t::set_geometry(const char*)':
background.C:273: error: invalid conversion from 'const char*' to 'char*'
background.C:274: error: invalid conversion from 'const char*' to 'char*'
background.C: In member function 'bool bgPixmap_t::set_file(const char*)':
background.C:680: error: invalid conversion from 'const char*' to 'char*'

Unsubscribing uus.

A fix for FTBFS from Fedora (the only link other than Gentoo BTS I could find is http://lists.schmorp.de/pipermail/rxvt-unicode/2009q2/000970.html)

A build log from karmic pbuilder

Karmic has been released so debdiff should go to karmic-proposed, also tweaked version.

A patch for Lucid Lynx

This bug was fixed in the package rxvt-unicode - 9.06-2ubuntu1

---------------
rxvt-unicode (9.06-2ubuntu1) lucid; urgency=low

  * apply patch to fix X "locking" on ctrl+middle mouse click (LP: #317366)
  * Fix FTBFS caused by changes in glibc 2.10
  * debian/control:
     - change Maintainer to Ubuntu MOTU Developers

rxvt-unicode (9.06-2) unstable; urgency=medium

  * Don't install an alternative for urxvtcd (due to #481123)
 -- Krzysztof Klimonda <email address hidden> Sun, 02 Aug 2009 13:43:36 +0200

ACK from MOTU-SRU for debdiff in #26.

Accepted into karmic-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Accepted rxvt-unicode into jaunty-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Note that urxvt upstream has rejected the patch used here to fix this issue as it is not a bug in urxvt but in the xserver.
I think it is a good-enough workaround for now (better than freezing), but once this bug is fixed in the xserver, this workaround should be removed.

The version in karmic-proposed (9.06-1ubuntu0.09.10.1) fixes this problem for me.

Confirming that the version in karmic-proposed builds from source.

This bug was fixed in the package rxvt-unicode - 9.06-1ubuntu0.09.10.1

---------------
rxvt-unicode (9.06-1ubuntu0.09.10.1) karmic-proposed; urgency=low

  * apply patch to fix X "locking" on ctrl+middle mouse click (LP: #317366)
  * Fix FTBFS caused by changes in glibc 2.10
  * debian/control:
    - change Maintainer to Ubuntu MOTU Developers
 -- Krzysztof Klimonda <email address hidden> Sun, 02 Aug 2009 13:43:36 +0200

Setting back to v-needed for the jaunty version.

Anyone who can test the proposed package? If not, the proposed update will be removed again. Thanks!

On 29 April 2010 19:27, Martin Pitt <email address hidden> wrote:
> Anyone who can test the proposed package? If not, the proposed update
> will be removed again. Thanks!

pitti, do you mean specifically this needs to be tested in jaunty-proposed?

--
Martin <http://launchpad.net/~mbp/>

@Martin: Right, since that's what this current SRU is aimed at. Thanks!

SRU Verification for Jaunty:
I have reproduced the problem with rxvt-unicode=9.06-1 and have verified that the version ofrxvt-unicode=9.06-1ubuntu0.09.04.1 in -proposed fixes the issue. No more hang with middle-click.

Marking as verification-done

This bug was fixed in the package rxvt-unicode - 9.06-1ubuntu0.09.04.1

---------------
rxvt-unicode (9.06-1ubuntu0.09.04.1) jaunty-proposed; urgency=low

  * apply patch to fix X "locking" on ctrl+middle mouse click (LP: #317366)
  * Fix FTBFS caused by changes in glibc 2.10
  * debian/control:
    - change Maintainer to Ubuntu MOTU Developers
 -- Krzysztof Klimonda <email address hidden> Sun, 02 Aug 2009 13:43:36 +0200