summaryrefslogtreecommitdiffstats
path: root/src/providers
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2012-10-31 17:58:22 +0100
committerJakub Hrozek <jhrozek@redhat.com>2012-11-01 11:08:21 +0100
commitd5f08686cad045fb46145ce765b7cef0d4530775 (patch)
treeeee94fc3e4a185e93d6f3f7646d93f3167927903 /src/providers
parentccc46e79a8c9c2d61c9351fc9f60b5bff8a99bfa (diff)
downloadsssd-d5f08686cad045fb46145ce765b7cef0d4530775.tar.gz
sssd-d5f08686cad045fb46145ce765b7cef0d4530775.tar.xz
sssd-d5f08686cad045fb46145ce765b7cef0d4530775.zip
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
Diffstat (limited to 'src/providers')
-rw-r--r--src/providers/ldap/sdap_async_groups.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/providers/ldap/sdap_async_groups.c b/src/providers/ldap/sdap_async_groups.c
index 60cbcea77..992678e98 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++;
}
}