From 843f1134d2a81de928ba4856f86cd9c64e8df40a Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek@redhat.com>
Date: Wed, 31 Oct 2012 17:58:22 +0100
Subject: LDAP: Fix off-by-one error when saving ghost users

The ldb_val's length parameter should not include the terminating NULL.
This was causing funky behaviour as the users were saved as binary
attributes.

https://fedorahosted.org/sssd/ticket/1614
---
 src/providers/ldap/sdap_async_groups.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

(limited to 'src/providers/ldap/sdap_async_groups.c')

diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index 60cbcea7..992678e9 100644
--- a/src/providers/ldap/sdap_async_groups.c
+++ b/src/providers/ldap/sdap_async_groups.c
@@ -406,7 +406,7 @@ static int sdap_save_group(TALLOC_CTX *memctx,
                 ret = ENOMEM;
                 goto fail;
             }
-            el->values[el->num_values].length = strlen(value.ptr)+1;
+            el->values[el->num_values].length = strlen(value.ptr);
             el->num_values++;
         }
     }
-- 
cgit