]> git.sur5r.net Git - openldap/commitdiff
ITS#7806 fixed memory leak in LDAPAsynConnection.cpp
authorKevin H. Patterson <kevpatt@khptech.com>
Wed, 26 Feb 2014 18:08:24 +0000 (13:08 -0500)
committerQuanah Gibson-Mount <quanah@openldap.org>
Tue, 1 Apr 2014 23:45:08 +0000 (16:45 -0700)
contrib/ldapc++/src/LDAPAsynConnection.cpp

index 894f8ba205a60dd852c0f3f07d53df3d80227b2d..582d2b53ae67113e618067fb5f3883ba207924d9 100644 (file)
@@ -43,7 +43,10 @@ LDAPAsynConnection::LDAPAsynConnection(const string& url, int port,
     this->setConstraints(cons);
 }
 
-LDAPAsynConnection::~LDAPAsynConnection(){}
+LDAPAsynConnection::~LDAPAsynConnection(){
+       unbind();
+       delete m_constr;
+}
 
 void LDAPAsynConnection::init(const string& hostname, int port){
     DEBUG(LDAP_DEBUG_TRACE,"LDAPAsynConnection::init" << endl);
@@ -51,6 +54,8 @@ void LDAPAsynConnection::init(const string& hostname, int port){
             "   hostname:" << hostname << endl
             << "   port:" << port << endl);
 
+       unbind();
+
     m_uri.setScheme("ldap");
     m_uri.setHost(hostname);
     m_uri.setPort(port);
@@ -66,6 +71,8 @@ void LDAPAsynConnection::init(const string& hostname, int port){
 }
 
 void LDAPAsynConnection::initialize(const std::string& uri){
+       unbind();
+
        m_uri.setURLString(uri);
     int ret = ldap_initialize(&cur_session, m_uri.getURLString().c_str());
     if ( ret != LDAP_SUCCESS ) {
@@ -277,6 +284,7 @@ void LDAPAsynConnection::unbind(){
 
 void LDAPAsynConnection::setConstraints(LDAPConstraints *cons){
     DEBUG(LDAP_DEBUG_TRACE,"LDAPAsynConnection::setConstraints()" << endl);
+       delete m_constr;
     m_constr=cons;
 }