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 766139 - Typing fills in password entry before user selection
Typing fills in password entry before user selection
Status: RESOLVED FIXED
Product: gnome-shell
Classification: Core
Component: login-screen
3.22.x
Other Linux
: Normal normal
: ---
Assigned To: Ray Strode [halfline]
gnome-shell-maint
: 755081 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2016-05-08 15:05 UTC by Bastien Nocera
Modified: 2017-02-15 11:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
loginDialog: don't allow type ahead at the login screen (1.92 KB, patch)
2016-05-09 13:01 UTC, Ray Strode [halfline]
committed Details | Review

Description Bastien Nocera 2016-05-08 15:05:06 UTC
After booting up my machine, I frequently forget that my session isn't locked, and will start typing my password. Instead of being ignored, the password entry is filled in.

It's a potential security issue, if one was to type their password without logging in, and that pre-typed password being used to log in as the user.
Comment 1 Ray Strode [halfline] 2016-05-09 13:01:23 UTC
well we should get rid of the screenshield entirely at the login screen which would fix this bug, but in the mean time, i'll attach a one-liner that should work.
Comment 2 Ray Strode [halfline] 2016-05-09 13:01:39 UTC
Created attachment 327517 [details] [review]
loginDialog: don't allow type ahead at the login screen

It's weird if after you select a username from the user list, there's
a password already filled in.

This commit disables at that feature for the login screen
(but keeps it in tact for the unlock screen)
Comment 3 Bastien Nocera 2016-05-09 13:12:33 UTC
(In reply to Ray Strode [halfline] from comment #1)
> well we should get rid of the screenshield entirely at the login screen
> which would fix this bug, but in the mean time, i'll attach a one-liner that
> should work.

How is this related to the screen shield? The problem happens when the screenshield isn't visible, and the user list visible.
Comment 4 Ray Strode [halfline] 2017-01-05 15:41:37 UTC
not sure I follow.  you're staring at a user list, and you start typing your password, realize you didn't mean to type your password, then click on your username, and the password is already filled in waiting for you?

The only way I could see that happening is if you started typing while the screenshield was done.
Comment 5 Bastien Nocera 2017-01-05 15:47:17 UTC
(In reply to Ray Strode [halfline] from comment #4)
> not sure I follow.  you're staring at a user list, and you start typing your
> password, realize you didn't mean to type your password, then click on your
> username, and the password is already filled in waiting for you?

Usually only a few characters from the password, not all of it. Not sure what else is eating key strokes.

> The only way I could see that happening is if you started typing while the
> screenshield was done.

The screenshield isn't there, the screen isn't in power save mode.
Comment 6 Ray Strode [halfline] 2017-01-05 16:05:00 UTC
are you typing the keys right at start up? or is it at any point you're staring at the login screen user list?
Comment 7 Bastien Nocera 2017-01-05 16:07:31 UTC
(In reply to Ray Strode [halfline] from comment #6)
> are you typing the keys right at start up? or is it at any point you're
> staring at the login screen user list?

Any point where I'm staring at the user list in gdm.
Comment 8 Bastien Nocera 2017-01-11 12:19:32 UTC
Couldn't reproduce with the latest updates, I'll reopen when I can.
Comment 9 Bastien Nocera 2017-02-10 18:21:04 UTC
You were right, it's the power save mode/screenshield.

1. Boot computer
2. Stay on login screen and wait for screensaver
3. Screen goes black but no screen blank (that'd be a separate bug)
4. Type your password
5. Select user
6. Some parts of the password entry is already filled in
Comment 10 Ray Strode [halfline] 2017-02-10 20:57:17 UTC
Can you try the patch?
Comment 11 Florian Müllner 2017-02-10 22:00:20 UTC
Review of attachment 327517 [details] [review]:

FWIW, the patch makes sense to me
Comment 12 Bastien Nocera 2017-02-14 02:06:11 UTC
Pushed to gnome-3-20 and gnome-3-22 as well

Attachment 327517 [details] pushed as 93071d9 - loginDialog: don't allow type ahead at the login screen
Comment 13 Sebastian Keller 2017-02-14 08:41:21 UTC
*** Bug 755081 has been marked as a duplicate of this bug. ***
Comment 14 Bastien Nocera 2017-02-15 11:11:29 UTC
(In reply to Bastien Nocera from comment #12)
> Pushed to gnome-3-20 and gnome-3-22 as well
> 
> Attachment 327517 [details] pushed as 93071d9 - loginDialog: don't allow
> type ahead at the login screen

I should note that I tested it!