summaryrefslogtreecommitdiff
path: root/docs-xml/manpages-3
diff options
context:
space:
mode:
authorSteven Danneman <sdanneman@sd-ubuntu.(none)>2008-04-25 18:34:46 -0700
committerJeremy Allison <jra@samba.org>2008-04-26 08:11:20 -0700
commit778a5414b1148ea767020b5330b076fed666694f (patch)
treed889aeb242d3326782b01f4954d337bc801e6eed /docs-xml/manpages-3
parent17ade782b9037500c282555e5a2612a863af58d8 (diff)
downloadsamba-778a5414b1148ea767020b5330b076fed666694f.tar.gz
samba-778a5414b1148ea767020b5330b076fed666694f.tar.bz2
samba-778a5414b1148ea767020b5330b076fed666694f.zip
Fix bug 5419: memory leak in ads_do_search_all_args() when enumerating 1000s of entries
The ads_do_search_all_args() function attempts to string together several LDAPMessage structures, returned across several paged ldap requests, into a single LDAPMessage structure. It does this by pulling entries off the second LDAPMessage structure and appending them to the first via the OpenLDAP specific ldap_add_result_entry() call. The problem with this approach is it skips non-entry messages such as the result, and controls. These messages are leaked. The short term solution as suggested by Volker is to replace the ads_*_entry() calls with ads_*_message() calls so we don't leak any messages. This fixes the leak but doesn't remove the dependence on the OpenLDAP specific implementation of ldap_add_result_entry(). (This used to be commit f1a5405409c396df394611e2a234522572d2860a)
Diffstat (limited to 'docs-xml/manpages-3')
0 files changed, 0 insertions, 0 deletions