-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117091/
-----------------------------------------------------------

Review request for kde-workspace and Aaron J. Seigo.


Bugs: 327947 and 329076
    http://bugs.kde.org/show_bug.cgi?id=327947
    http://bugs.kde.org/show_bug.cgi?id=329076


Repository: kde-workspace


Description
-------

If the screen locker is set to not require a password to unlock, it will not 
show the password input field even when the powermanagement settings suspend 
the system and are set to require a password after resume (when it was already 
running at that point).
This locks people out of their system.

As there seems to be no way to switch the already running greeter to 
immediateLock mode, it is killed in that case by this patch. It gets restarted 
immediately with the --immediatelock option in KSldApp::lockProcessFinished() 
and shows the password input field then.


Diffs
-----

  ksmserver/screenlocker/ksldapp.cpp 3dfcc9e 

Diff: https://git.reviewboard.kde.org/r/117091/diff/


Testing
-------

Disable "Require password after" in the screen locker settings (the default), 
set it to start after 1 min. (for easier testing).
Enable "Suspend session after" and set it to 2 minutes. (set the action to 
"Suspend", "Hibernate", or "Lock Screen", doesn't matter)
Make sure "Lock screen on resume" is enabled in the powermanagements "Advanced 
Options" (it is by default).

After 1 minute the screen locker kicks in, and doesn't require a password.
After 2 minutes the session gets suspended, hibernated or locked, and requires 
a password to resume.

Without this patch no password dialog is shown, the user cannot resume the 
session by entering the password.

With this patch this works: there is a password input field, the session is 
unlocked when the user enters the password.

Other openSUSE users have tested this as well, see f.e. 
https://bugzilla.novell.com/show_bug.cgi?id=864305#c4


Thanks,

Wolfgang Bauer

Reply via email to