Package: ssh-askpass
Version: 1:1.2.4.1-7
Severity: normal

Hi Philip,

x11-ssh-askpass sets the TRANSIENT_FOR X property to itself:

    $ { xwininfo ; xprop ; } | grep -i 'window id'
    xwininfo: Window id: 0xc00009 "OpenSSH Authentication Passphrase Request"
    WM_TRANSIENT_FOR(WINDOW): window id # 0xc00009

The ICCCM says, (emphasis mine)

    "4.1.2.6. WM_TRANSIENT_FOR Property 

    The WM_TRANSIENT_FOR property (of type WINDOW) contains the ID of
    *another* top-level window. The implication is that this window is a
    pop-up on behalf of the named window, and window managers may decide
    not to decorate transient windows or may *treat them differently* in
    other ways."

        http://tronche.com/gui/x/icccm/sec-4.html#s-4.1.2.6

While attempting to treat x11-ssh-askpass differently because it set
WM_TRANSIENT_FOR, the Awesome window manager goes into an infinite loop
looking for x11-ssh-askpass's parent. See Debian bug#529025:

    http://bugs.debian.org/529025

Admittedly, Awesome should protect itself from infinite loops, but
x11-ssh-askpass shouldn't set the WM_TRANSIENT_FOR property to itself
unless it must. Unfortunately, x11-ssh-askpass.c is sparsely commented
and doesn't contain any justification for setting WM_TRANSIENT_FOR.
Gnome-ssh-askpass (package: ssh-askpass-gnome) doesn't set the property,
so I suspect it isn't necessary.

If possible, please disable the call to XSetTransientForHint on
x11-ssh-askpass.c line number 803 to improve compatibility with
the ICCCM.

Thank you,

-Dave
-- 
David A. Harding            Website:  http://dtrt.org/
1 (609) 997-0765              Email:  [email protected]
                        Jabber/XMPP:  [email protected]



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to