This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/4460/

On December 23rd, 2010, 8:09 p.m., Martin Gräßlin wrote:

Do you have an svn account? Trunk is open again, so this can be committed.

Unfortunately I cannot test it as my KAuth setup is something broken.
I don't have an svn account. 
I'll test the patch again after christmas and 
ping this review request when I've verified it

- Mikael


On June 26th, 2010, 3:05 p.m., Mikael Gerdin wrote:

Review request for kwin.
By Mikael Gerdin.

Updated 2010-06-26 15:05:01

Description 

Makes KWin's window killer able to kill processes running as other users.
I just re-used the code used in krunner's (and ksysguard's) process lister
to kill processes using elevated privileges.

As it works now it only tries to use elevated privileges if kill returns EPERM.

Testing 

Manual testing by calling the helper app manually and by making a
root-owned X window hang and click the "X" to make kwin try to kill it.
Bugs: 129476

Diffs

  • /trunk/KDE/kdebase/workspace/kwin/killer/killer.cpp (1143106)

View Diff