[PATCH] ldap: use case insensitive check

[Arjun Sreedharan]

Signed-off-by: Arjun Sreedharan <arjun024(a)gmail.com&gt;
---
 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

[Jakub Hrozek]

On Mon, Dec 15, 2014 at 01:15:31PM +0530, Arjun Sreedharan wrote:
>  Signed-off-by: Arjun Sreedharan <arjun024(a)gmail.com&gt;
>  ---
>   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.