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 333340 - Hitting enter before the lock dialog comes up causes it to try logging in with an empty password
Hitting enter before the lock dialog comes up causes it to try logging in wit...
Status: RESOLVED FIXED
Product: gnome-screensaver
Classification: Deprecated
Component: general
2.13.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-screensaver maintainers
gnome-screensaver maintainers
Depends on:
Blocks:
 
 
Reported: 2006-03-04 07:40 UTC by Christopher Aillon
Modified: 2006-03-06 19:53 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (1.35 KB, patch)
2006-03-06 18:35 UTC, William Jon McCann
committed Details | Review

Description Christopher Aillon 2006-03-04 07:40:37 UTC
Upstreaming from: https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=177598

Description of problem:

Hitting enter before the dialog pops up sends the event anyway and the dialog
sits for a few seconds before it allows you to enter a new password

How reproducible:

Always

Steps to Reproduce:
1. Lock the screen
2. hit enter
  
Actual results:

Dialog blocks for a few seconds before you can enter another password

Expected results:

Key events should work the same as in Xscreensaver.  The first enter is ignored
and just brings up the dialog if it is not showing.  All other keystrokes are
valid even if the dialog is hidden.  An enter after any other keystroke should
also be valid.

Additional Info:

The first space is also invalid.  I still have Xscreensaver on my desktop if you
want to take a look.
Comment 1 William Jon McCann 2006-03-06 18:35:42 UTC
Created attachment 60780 [details] [review]
patch

Unfortunately, we just hit hard code freeze today.  So, I guess this will have to wait.
Comment 2 William Jon McCann 2006-03-06 18:47:50 UTC
Hey Vincent, can I get approval to commit this one liner?
Comment 3 Vincent Untz 2006-03-06 19:49:24 UTC
You don't need approval yet: the freeze starts at midnight UTC. Still 4 hours left ;-)
Comment 4 William Jon McCann 2006-03-06 19:53:25 UTC
Cool - off by one I guess.  Committed.

2006-03-06  William Jon McCann  <mccann@jhu.edu>

	* src/gs-window-x11.c (queue_key_event): Don't
	queue keypad enter presses.  Fixes #333340