diff options
author | Rich Megginson <rmeggins@redhat.com> | 2008-08-27 21:05:30 +0000 |
---|---|---|
committer | Rich Megginson <rmeggins@redhat.com> | 2008-08-27 21:05:30 +0000 |
commit | b6b7f5e4bc4701a9a50f43578dfbc7e1cb300a02 (patch) | |
tree | 730768ebcdd14b2b2481e767da9de13634cef926 /lib/libaccess/ldapacl.cpp | |
parent | b32761a5797fa5efb8ab65ae2d2e0c539357e73c (diff) | |
download | ds-b6b7f5e4bc4701a9a50f43578dfbc7e1cb300a02.tar.gz ds-b6b7f5e4bc4701a9a50f43578dfbc7e1cb300a02.tar.xz ds-b6b7f5e4bc4701a9a50f43578dfbc7e1cb300a02.zip |
Resolves: bug 458510
Bug Description: Memory leak setting password with passwd extop
Reviewed by: nkinder (Thanks!)
Branch: HEAD
Fix Description: 1) if the given dn is "", that 1 byte will be leaked when the dn is reassigned
to the bind dn - so free it first in that case before reassigning
2) calling slapi_pblock_get with SLAPI_CONN_DN does a strdup, which is
different than most uses of slapi_pblock_get. That memory must be freed. So we free it at the end.
3) If we set the ORIGINAL_TARGET to a dn other than the given dn, we must free
it - grab it and compare it to dn - if not the same, free dn first, then free the original target dn
Platforms tested: RHEL5, Fedora 8
Flag Day: no
Doc impact: no
QA impact: should be covered by regular nightly and manual testing
New Tests integrated into TET: none
Diffstat (limited to 'lib/libaccess/ldapacl.cpp')
0 files changed, 0 insertions, 0 deletions