Skip to content

Commit

Permalink
Remove a check that was left behind
Browse files Browse the repository at this point in the history
When refactoring talloc_asprintf calls a check was left behind that
cased the backend to go offline immediately.
  • Loading branch information
jhrozek authored and sgallagher committed Feb 25, 2010
1 parent af81aaa commit 52e080e
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions src/providers/ldap/ldap_id_cleanup.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand All @@ -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)))",
Expand Down

0 comments on commit 52e080e

Please sign in to comment.