0010560: Connection leak in AcctServer if anything goes wrong

Index: src/org/openbravo/erpCommon/ad_forms/AcctServer.java
===================================================================
--- src/org/openbravo/erpCommon/ad_forms/AcctServer.java	(revision 14788)
+++ src/org/openbravo/erpCommon/ad_forms/AcctServer.java	(revision 14789)
@@ -263,8 +263,9 @@
      */
     public void run(VariablesSecureApp vars, boolean runAsBatch) throws IOException, ServletException{
       if (AD_Client_ID.equals("")) AD_Client_ID = vars.getClient();
+      Connection con = null;
       try {
-        Connection con = connectionProvider.getTransactionConnection();
+        con = connectionProvider.getTransactionConnection();
         String strIDs = "";
         if (log4j.isDebugEnabled()) log4j.debug("AcctServer - Run - TableName = " + tableName);
         boolean morePendingDocuments = true;
@@ -294,11 +295,21 @@
           con.commit();
         }
         connectionProvider.releaseCommitConnection(con);
+        con = null;
       } catch (NoConnectionAvailableException ex) {
         throw new ServletException("@CODE=NoConnectionAvailable");
       } catch (SQLException ex2) {
         throw new ServletException("@CODE=" + Integer.toString(ex2.getErrorCode()) + "@" + ex2.getMessage());
       }
+      finally {
+    	  if (con!=null)
+			try {
+				connectionProvider.releaseRollbackConnection(con);
+				con=null;
+			} catch (SQLException e) {
+				log4j.warn("Exception in releasing connection", e);
+			}
+      }
     }
 
     /**