You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2011/09/03 14:43:28 UTC

svn commit: r1164858 - /directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java

Author: seelmann
Date: Sat Sep  3 12:43:27 2011
New Revision: 1164858

URL: http://svn.apache.org/viewvc?rev=1164858&view=rev
Log:
Close cursor

Modified:
    directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java

Modified: directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java
URL: http://svn.apache.org/viewvc/directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java?rev=1164858&r1=1164857&r2=1164858&view=diff
==============================================================================
--- directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java (original)
+++ directory/apacheds/trunk/core/src/main/java/org/apache/directory/server/core/subtree/SubentryInterceptor.java Sat Sep  3 12:43:27 2011
@@ -478,7 +478,6 @@ public class SubentryInterceptor extends
         {
             if ( aps.next() )
             {
-                aps.close();
                 return true;
             }
         }
@@ -486,7 +485,17 @@ public class SubentryInterceptor extends
         {
             throw new LdapOperationException( e.getMessage(), e );
         }
-
+        finally
+        {
+            try
+            {
+                aps.close();
+            }
+            catch ( Exception e )
+            {
+                LOG.error( I18n.err( I18n.ERR_168 ), e );
+            }
+        }
 
         return false;
     }