After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 349059 - bug with rdesktop
bug with rdesktop
Status: RESOLVED FIXED
Product: gnome-screensaver
Classification: Deprecated
Component: general
2.14.x
Other Linux
: Normal major
: ---
Assigned To: gnome-screensaver maintainers
gnome-screensaver maintainers
Depends on:
Blocks:
 
 
Reported: 2006-07-28 09:35 UTC by rsalmon
Modified: 2006-11-03 16:22 UTC
See Also:
GNOME target: ---
GNOME version: 2.13/2.14


Attachments
debug log from HEAD (30.52 KB, text/plain)
2006-08-29 15:05 UTC, William Jon McCann
  Details
debug log from 2.14.4 (gzipped) (24.40 KB, application/x-gzip)
2006-08-29 15:18 UTC, William Jon McCann
  Details
sync and flush after nuke focus (554 bytes, patch)
2006-08-29 16:19 UTC, William Jon McCann
committed Details | Review

Description rsalmon 2006-07-28 09:35:11 UTC
if the mouse is over a rdesktop session when gnome-screensaver kicks off, gnome-screensaver :
- doesn't always lock the screen (prompt for a password)
- mouse and keyboard are working
- everything seams to work but I can't open/create new windows (eg: If I start xterm, won't work)
- I need to kill X11 (CTRL+ALT+DEL) because I can't open any new windows/apps.

if the mouse isn't over a rdesktop session, everything seams fine.

weird!
I wonder if there is a problem between the MS Windows screensaver, that eventually kicks off within the rdesktop session, and gnome-screensaver.


Ronan.
Comment 1 William Jon McCann 2006-07-31 15:46:28 UTC
Sounds like rdesktop is grabbing the keyboard.  For the screensaver to lock correctly it must grab (and maintain) control over the keyboard.  If it cannot do this then it tries a few times.

What version of gnome-screensaver and rdesktop do you have?
Comment 2 rsalmon 2006-08-01 08:14:48 UTC
(In reply to comment #1)
> Sounds like rdesktop is grabbing the keyboard.  For the screensaver to lock
> correctly it must grab (and maintain) control over the keyboard.  If it cannot
> do this then it tries a few times.
> 
> What version of gnome-screensaver and rdesktop do you have?
> 

Sorry, forgot to include versions :-(

Name        : gnome-screensaver            Relocations: (not relocatable)
Version     : 2.14.2                            Vendor: Red Hat, Inc.
Release     : 1.fc5.1                       

Name        : rdesktop                     Relocations: (not relocatable)
Version     : 1.4.1                             Vendor: Red Hat, Inc.
Release     : 3.2.1                         Build Date: dim 12 fév 2006 10:06:59

Comment 3 Daniel Holbach 2006-08-29 12:59:13 UTC
Mentioned in https://launchpad.net/distros/ubuntu/+source/gnome-screensaver/+bug/58093 as well.
Comment 4 William Jon McCann 2006-08-29 15:05:50 UTC
Created attachment 71835 [details]
debug log from HEAD

Can't reproduce with HEAD.  However, it is strange that our grabbing fails for the warning fade but not for the activation itself.

Will try 2-14 next.
Comment 5 William Jon McCann 2006-08-29 15:18:27 UTC
Created attachment 71836 [details]
debug log from 2.14.4 (gzipped)

With 2.14.4 the warning fade occurs (different bug) but the lock fails then the screensaver resets and tries again.  The next time the lock succeeds.  I'm unable to unlock the screen and everything seems fine.  So I guess I'm unable to reproduce this bug.
Comment 6 William Jon McCann 2006-08-29 16:19:07 UTC
Created attachment 71841 [details] [review]
sync and flush after nuke focus

Any chance this patch improves things for you?

BTW, I have rdesktop-1.4.1-3.2.1.  This version seems to give up the grab after nuking the focus which is simply:

        XGetInputFocus (GDK_DISPLAY (), &focus, &rev);
        XSetInputFocus (GDK_DISPLAY (), None, RevertToNone, CurrentTime);
Comment 7 William Jon McCann 2006-08-29 16:36:17 UTC
rdesktop seems to ungrab on FocusOut:
http://rdesktop.cvs.sourceforge.net/rdesktop/rdesktop/xwin.c?revision=1.213&view=markup
Comment 8 William Jon McCann 2006-10-25 17:14:36 UTC
That patch had been committed a while back.
Comment 9 rsalmon 2006-11-03 16:22:07 UTC
Seems to work fine on Fedora core 6 with the following versions : 

Name        : rdesktop                     Relocations: (not relocatable)
Version     : 1.4.1                             Vendor: Red Hat, Inc.
Release     : 4                             Build Date: jeu 31 aoû 2006 19:26:15 Name        : gnome-screensaver            Relocations: (not relocatable)
Version     : 2.16.0                            Vendor: Red Hat, Inc.
Release     : 7.fc6                         Build Date: dim 01 oct 2006 23:35:37 


Thanks again,
Ronan