Close searchResults so LDAP context can close.

Fixes #489

Author: perry2of5

core/src/main/java/org/springframework/ldap/pool/validation/DefaultDirContextValidator.java

@@ -165,9 +165,10 @@ public void setSearchControls(SearchControls searchControls) {
     public boolean validateDirContext(DirContextType contextType, DirContext dirContext) {
         Assert.notNull(contextType, "contextType may not be null");
         Assert.notNull(dirContext, "dirContext may not be null");
-        
+
+        NamingEnumeration<SearchResult> searchResults = null;        
         try {
-            final NamingEnumeration<SearchResult> searchResults = dirContext.search(this.base, this.filter, this.searchControls);
+            searchResults = dirContext.search(this.base, this.filter, this.searchControls);
 
             if (searchResults.hasMore()) {
                 this.logger.debug("DirContext '{}' passed validation.", dirContext);
@@ -179,6 +180,11 @@ public boolean validateDirContext(DirContextType contextType, DirContext dirCont
             this.logger.debug("DirContext '{}' failed validation with an exception.", dirContext, e);
             return false;
         }
+        finally {
+            if (searchResults != null) {
+                searchResults.close();
+            }
+        }
 
         this.logger.debug("DirContext '{}' failed validation.", dirContext);
         return false;

core/src/main/java/org/springframework/ldap/pool2/validation/DefaultDirContextValidator.java

@@ -166,8 +166,9 @@ public boolean validateDirContext(DirContextType contextType, DirContext dirCont
         Assert.notNull(contextType, "contextType may not be null");
         Assert.notNull(dirContext, "dirContext may not be null");
 
+        NamingEnumeration<SearchResult> searchResults = null;
         try {
-            final NamingEnumeration<SearchResult> searchResults = dirContext.search(this.base, this.filter, this.searchControls);
+            searchResults = dirContext.search(this.base, this.filter, this.searchControls);
 
             if (searchResults.hasMore()) {
                 this.logger.debug("DirContext '{}' passed validation.", dirContext);
@@ -179,6 +180,11 @@ public boolean validateDirContext(DirContextType contextType, DirContext dirCont
             this.logger.debug("DirContext '{}' failed validation with an exception.", dirContext, e);
             return false;
         }
+        finally {
+            if (searchResults != null) {
+                searchResults.close();
+            }
+        }
 
         this.logger.debug("DirContext '{}' failed validation.", dirContext);
         return false;