summaryrefslogtreecommitdiffstats
path: root/src/providers
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2011-04-28 17:21:32 +0200
committerStephen Gallagher <sgallagh@redhat.com>2011-04-28 13:20:01 -0400
commit8c7e6e71b435857802f12c8de09d34b052880eb6 (patch)
tree905f30c7b52872cc6e2a0e99700cec76ff866ec3 /src/providers
parent333b7970cc60c6277363c80564456a716c2d6634 (diff)
downloadsssd-8c7e6e71b435857802f12c8de09d34b052880eb6.tar.gz
sssd-8c7e6e71b435857802f12c8de09d34b052880eb6.tar.xz
sssd-8c7e6e71b435857802f12c8de09d34b052880eb6.zip
Do not leak LDAP paging controls
Diffstat (limited to 'src/providers')
-rw-r--r--src/providers/ldap/sdap_async.c5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/providers/ldap/sdap_async.c b/src/providers/ldap/sdap_async.c
index 5e05cb742..1e083553d 100644
--- a/src/providers/ldap/sdap_async.c
+++ b/src/providers/ldap/sdap_async.c
@@ -921,6 +921,8 @@ static errno_t sdap_get_generic_step(struct tevent_req *req)
state->scope, state->filter,
discard_const(state->attrs),
false, m_controls, NULL, NULL, 0, &msgid);
+ ldap_control_free(page_control);
+ m_controls[0] = NULL;
if (lret != LDAP_SUCCESS) {
DEBUG(3, ("ldap_search_ext failed: %s\n", ldap_err2string(lret)));
if (lret == LDAP_SERVER_DOWN) {
@@ -1038,6 +1040,7 @@ static void sdap_get_generic_done(struct sdap_op *op,
lret = ldap_parse_pageresponse_control(state->sh->ldap, page_control,
&total_count, &cookie);
+ ldap_controls_free(returned_controls);
if (lret != LDAP_SUCCESS) {
DEBUG(1, ("Could not determine page control"));
tevent_req_error(req, EIO);
@@ -1068,6 +1071,8 @@ static void sdap_get_generic_done(struct sdap_op *op,
return;
}
+ /* The cookie must be freed even if len == 0 */
+ ber_memfree(cookie.bv_val);
/* This was the last page. We're done */