-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
This behavior was present for getpwnam() but was lacking for initgroups.
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=677768
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
On Thu, Feb 17, 2011 at 03:49:17PM -0500, Stephen Gallagher wrote:
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
This behavior was present for getpwnam() but was lacking for initgroups.
this patch works as expected, so
ACK
While testing I tried the same with a group, i.e. log in and off as a user, add user to a group on the server, log in check groupmembership and log off again, remove group from the server, log in check groupmembership. The result was that the user is still a member of the group although the group was deleted on the server. Maybe you want to fix this in the same patch, because it looks related.
bye, Sumit
Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/ -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/
iEYEARECAAYFAk1dic0ACgkQeiVVYja6o6O9pgCfUvi2N0BdrM0qXHPovp/CO9SU r84AoIhkmG7dTlWlUbysUSnwo/cs5e2r =ZZgh -----END PGP SIGNATURE-----
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1
On 02/18/2011 07:05 AM, Sumit Bose wrote:
On Thu, Feb 17, 2011 at 03:49:17PM -0500, Stephen Gallagher wrote: This behavior was present for getpwnam() but was lacking for initgroups.
Resolves https://bugzilla.redhat.com/show_bug.cgi?id=677768
this patch works as expected, so
ACK
While testing I tried the same with a group, i.e. log in and off as a user, add user to a group on the server, log in check groupmembership and log off again, remove group from the server, log in check groupmembership. The result was that the user is still a member of the group although the group was deleted on the server. Maybe you want to fix this in the same patch, because it looks related.
Pushed to master and sssd-1-5.
We discussed off-list and the group issue was not related, so it will get its own patch.
- -- Stephen Gallagher RHCE 804006346421761
Delivering value year after year. Red Hat ranks #1 in value among software vendors. http://www.redhat.com/promo/vendor/
sssd-devel@lists.fedorahosted.org