diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2010-02-25 14:23:52 +0100 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2010-02-25 08:40:27 -0500 |
| commit | 52e080ee2d50b6439b503695fb12808b3d0cc30b (patch) | |
| tree | 4b40969b675ed1ee8451ec31a169f0183e71422d /src/providers/ldap/ldap_id_cleanup.c | |
| parent | af81aaa57f82eab78647113c391bd84247f96150 (diff) | |
| download | sssd-52e080ee2d50b6439b503695fb12808b3d0cc30b.tar.gz sssd-52e080ee2d50b6439b503695fb12808b3d0cc30b.tar.bz2 sssd-52e080ee2d50b6439b503695fb12808b3d0cc30b.zip | |
Remove a check that was left behind
When refactoring talloc_asprintf calls a check was left behind that
cased the backend to go offline immediately.
Diffstat (limited to 'src/providers/ldap/ldap_id_cleanup.c')
| -rw-r--r-- | src/providers/ldap/ldap_id_cleanup.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/src/providers/ldap/ldap_id_cleanup.c b/src/providers/ldap/ldap_id_cleanup.c index 64de5403..53c9501f 100644 --- a/src/providers/ldap/ldap_id_cleanup.c +++ b/src/providers/ldap/ldap_id_cleanup.c @@ -277,7 +277,7 @@ static struct tevent_req *cleanup_users_send(TALLOC_CTX *memctx, struct cleanup_users_state *state; static const char *attrs[] = { SYSDB_NAME, SYSDB_UIDNUM, NULL }; time_t now = time(NULL); - char *subfilter; + char *subfilter = NULL; int account_cache_expiration; req = tevent_req_create(memctx, &state, struct cleanup_users_state); @@ -298,12 +298,6 @@ static struct tevent_req *cleanup_users_send(TALLOC_CTX *memctx, DEBUG(9, ("Cache expiration is set to %d days\n", account_cache_expiration)); - if (!subfilter) { - DEBUG(2, ("Failed to build filter\n")); - talloc_zfree(req); - return NULL; - } - if (account_cache_expiration > 0) { subfilter = talloc_asprintf(state, "(&(!(%s=0))(%s<=%ld)(|(!(%s=*))(%s<=%ld)))", |