Review Request 117644: screenlocker: don't leave behind screensaver processes

[Commit Hook]

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


This review has been submitted with commit 26a8ddc018d940ea1910ced6264344b520ad8405 by Wolfgang Bauer to branch KDE/4.11.

- Commit Hook


On April 24, 2014, 1:42 p.m., Wolfgang Bauer wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117644/
> -----------------------------------------------------------
> 
> (Updated April 24, 2014, 1:42 p.m.)
> 
> 
> Review request for kde-workspace and Plasma.
> 
> 
> Bugs: 224200
>     http://bugs.kde.org/show_bug.cgi?id=224200
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> Currently the screen locker just kills the greeter (kscreenlocker_greet) when the screen is unlocked by the user during the grace time.
> But apparently this can leave behind running screensaver processes launched by the greeter, see the bug report (which has the highest number of  votes of all open bugs AFAICS).
> 
> This patch changes this to only terminate the greeter, and adds a signal handler to the greeter to exit gracefully in this case.
> The signal handler exits with return code 1, so that it is not possible to circumvent the password input by just sending a SIGTERM. (the screen locker restarts the greeter in case it doesn't quit with exit code 0)
> 
> 
> Diffs
> -----
> 
>   ksmserver/screenlocker/greeter/main.cpp d898734 
>   ksmserver/screenlocker/ksldapp.cpp 3dfcc9e 
> 
> Diff: https://git.reviewboard.kde.org/r/117644/diff/
> 
> 
> Testing
> -------
> 
> Configure a legacy screensaver in Systemsettings->Display and Monitor->Screen Locker, be sure to leave "Require Password after" disabled.
> Wait for the screen locker to kick in.
> Unlock the screen by moving the mouse or pressing a key.
> Check the process list.
> 
> Without this patch at least kswarm.kss and kblankscreen.kss reliably kept running after unlocking the screen on my system.
> With this patch they quit themselves.
> 
> I'm using this patch for over two weeks now, and I haven't seen any left-over screen saver processes any more (and I even set the timeout to 1 minute).
> 
> I also tried to terminate kscreenlocker_greet manually by running "killall kscreenlocker_greet" from a text console in case of a password required, and the locker didn't quit, you still have to enter the password.
> 
> 
> Thanks,
> 
> Wolfgang Bauer
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140424/12c15270/attachment.htm>