[SSSD] [PATCHES] sudo: include primary group in user group list

Jakub Hrozek jhrozek at redhat.com
Thu Dec 13 19:21:38 UTC 2012


On Thu, Dec 13, 2012 at 12:40:28PM +0100, Pavel Březina wrote:
> https://fedorahosted.org/sssd/ticket/1677
> 
> 0001:
> just a very little refactoring when touching the function
> 

Ack

> 0002:
> Resolve primary group in addition to secondary groups, so we don't
> longer require memberUid present in the primary group.
> 

Ack

> However, if the memberUid is present, it may happen that the group
> is included in the list twice, which means that it will be included
> in the sysdb filter twice. I think this to be a better solution than
> to go over all secondary groups and test whether it is also user's
> primary group.

I think this is fine.



More information about the sssd-devel mailing list