[PATCH] Use sysdb attribute name for GID, not LDAP attribute

[Stephen Gallagher]

This was working in the common case, because for most (non-AD) directory
servers, the LDAP attribute and the sysdb attribute had the same name.

However, this was incorrect behavior.

In the future, I've asked our QE group to set up LDAP servers with ALL
non-standard attribute names, so that we'll discover mistakes of this
nature sooner.

[Simo Sorce]

On Thu, 2011-08-11 at 13:14 -0400, Stephen Gallagher wrote:
>  This was working in the common case, because for most (non-AD)
> directory
>  servers, the LDAP attribute and the sysdb attribute had the same name.
>  
>  However, this was incorrect behavior.
>  
>  In the future, I've asked our QE group to set up LDAP servers with ALL
>  non-standard attribute names, so that we'll discover mistakes of this
>  nature sooner. 
Obviously correct, ACK.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

[Stephen Gallagher]

On Thu, 2011-08-11 at 13:16 -0400, Simo Sorce wrote:
>  On Thu, 2011-08-11 at 13:14 -0400, Stephen Gallagher wrote:
>  > This was working in the common case, because for most (non-AD) directory
>  > servers, the LDAP attribute and the sysdb attribute had the same name.
>  > 
>  > However, this was incorrect behavior.
>  > 
>  > In the future, I've asked our QE group to set up LDAP servers with ALL
>  > non-standard attribute names, so that we'll discover mistakes of this
>  > nature sooner.
>  
>  Obviously correct, ACK.
>  
>  Simo.
>   
Pushed to master, sssd-1-5 and sssd-1-6.