diff options
| author | Simo Sorce <ssorce@redhat.com> | 2009-03-10 01:28:14 -0400 |
|---|---|---|
| committer | Simo Sorce <ssorce@redhat.com> | 2009-03-10 01:28:14 -0400 |
| commit | 931eca3051c6607014d5df45c35115fb1234ab45 (patch) | |
| tree | 3ae85ee8b1f183e0bec18284b5faed50b3ed939d /server/db | |
| parent | 68303b58587b0e82efb1c57d96e89578e5290867 (diff) | |
| download | sssd-931eca3051c6607014d5df45c35115fb1234ab45.tar.gz sssd-931eca3051c6607014d5df45c35115fb1234ab45.tar.xz sssd-931eca3051c6607014d5df45c35115fb1234ab45.zip | |
Fix bugs in functions dealing with groups
Fix infinite loop within initgr functions.
Fix min length check copy&paste error, was filtering valid groups if the name
was short enough and the group had no members.
Diffstat (limited to 'server/db')
| -rw-r--r-- | server/db/sysdb_search.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/server/db/sysdb_search.c b/server/db/sysdb_search.c index 15a7d4574..2a272cd7c 100644 --- a/server/db/sysdb_search.c +++ b/server/db/sysdb_search.c @@ -567,6 +567,9 @@ static void initgr_mem_legacy(struct sysdb_search_ctx *sctx) return request_ldberror(sctx, LDB_ERR_OPERATIONS_ERROR); } + /* make sure we don't loop with get_gen_callback() */ + sctx->gen_aux_fn = NULL; + userid = ldb_msg_find_attr_as_string(res->msgs[0], SYSDB_NAME, NULL); if (!userid) { return request_ldberror(sctx, LDB_ERR_OPERATIONS_ERROR); @@ -616,6 +619,9 @@ static void initgr_mem_search(struct sysdb_search_ctx *sctx) return request_ldberror(sctx, LDB_ERR_OPERATIONS_ERROR); } + /* make sure we don't loop with get_gen_callback() */ + sctx->gen_aux_fn = NULL; + sctx->expression = talloc_asprintf(sctx, SYSDB_INITGR_FILTER); if (!sctx->expression) { return request_ldberror(sctx, LDB_ERR_OPERATIONS_ERROR); |