summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2010-02-25 14:23:52 +0100
committerStephen Gallagher <sgallagh@redhat.com>2010-02-25 08:40:27 -0500
commit52e080ee2d50b6439b503695fb12808b3d0cc30b (patch)
tree4b40969b675ed1ee8451ec31a169f0183e71422d
parentaf81aaa57f82eab78647113c391bd84247f96150 (diff)
downloadsssd-52e080ee2d50b6439b503695fb12808b3d0cc30b.tar.gz
sssd-52e080ee2d50b6439b503695fb12808b3d0cc30b.tar.xz
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.
-rw-r--r--src/providers/ldap/ldap_id_cleanup.c8
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 64de54033..53c9501ff 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)))",