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

[Pavel Březina]

https://fedorahosted.org/sssd/ticket/1677

0001:
just a very little refactoring when touching the function

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

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.

[Jakub Hrozek]

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.

[Jakub Hrozek]

On Thu, Dec 13, 2012 at 08:21:38PM +0100, Jakub Hrozek wrote:
> 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.

Pushed to master and sssd-1-9