[SSSD] [PATCH] Make sss_userdel check for logged in users

Jakub Hrozek jhrozek at redhat.com
Tue Apr 6 16:03:50 UTC 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 04/06/2010 04:34 PM, Stephen Gallagher wrote:
> Sorry, I hate to say this, but I realized that killall returns nonzero
> if there are no processes running by that user name. So this warning is
> going to look wrong if we run the userdel command on a system where the
> user isn't logged in. I withdraw my request for this warning.
> 
> Also, we probably need to do 'killall -9' rather than simply killall, as
> SIGTERM doesn't terminate shell processes (which are the most important
> ones to kill if we need to kick a user)

Ah, correct. I only tested with ssh session where the kick worked fine.

Thank you for the review, a new patch is attached.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAku7W2YACgkQHsardTLnvCUBSACZATLBa5vYZqlX68uj49KouLcw
uOgAoI7d/7VoXxtoI9Cb9hGc4/cwUCub
=2f+g
-----END PGP SIGNATURE-----
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Make-sss_userdel-check-for-logged-in-users.patch
Type: text/x-patch
Size: 7005 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20100406/9a4ec0ee/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Make-sss_userdel-check-for-logged-in-users.patch.sig
Type: application/pgp-signature
Size: 72 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20100406/9a4ec0ee/attachment.sig>


More information about the sssd-devel mailing list