[SSSD] [PATCH] ldap: use case insensitive check

Jakub Hrozek jhrozek at redhat.com
Mon Jan 5 10:42:43 UTC 2015


On Mon, Dec 15, 2014 at 01:15:31PM +0530, Arjun Sreedharan wrote:
> Signed-off-by: Arjun Sreedharan <arjun024 at gmail.com>
> ---
>  src/providers/ldap/ldap_id_cleanup.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/providers/ldap/ldap_id_cleanup.c b/src/providers/ldap/ldap_id_cleanup.c
> index f78d4b4..88e487b 100644
> --- a/src/providers/ldap/ldap_id_cleanup.c
> +++ b/src/providers/ldap/ldap_id_cleanup.c
> @@ -361,7 +361,7 @@ static int cleanup_groups(TALLOC_CTX *memctx,
>          }
>  
>          posix = ldb_msg_find_attr_as_string(msgs[i], SYSDB_POSIX, NULL);
> -        if (!posix || strcmp(posix, "TRUE") == 0) {
> +        if (!posix || strcasecmp(posix, "TRUE") == 0) {
>              /* Search for users that are members of this group, or
>               * that have this group as their primary GID.
>               * Include subdomain users as well.
> -- 
> 1.7.11.7

What is the problem this patch is trying to solve? IIRC we only store
TRUE in the database.



More information about the sssd-devel mailing list