Merge pull request #405 from pbatey/master

destroy socket if auto-bind fails

Author: jsumners

lib/client/client.js

@@ -348,6 +348,9 @@ function Client(options) {
     this.on('setup', function (clt, cb) {
       clt.bind(options.bindDN, options.bindCredentials, function (err) {
         if (err) {
+          if (self._socket) {
+            self._socket.destroy()
+          }
           self.emit('error', err);
         }
         cb(err);

test/client.test.js

@@ -334,6 +334,7 @@ test('auto-bind bad credentials', function (t) {
   });
   clt.once('error', function (err) {
     t.equal(err.code, ldap.LDAP_INVALID_CREDENTIALS);
+    t.ok(clt._socket.destroyed, 'expect socket to be destroyed');
     clt.destroy();
     t.end();
   });