IDEMPIERE-5994 Connection leak when using DB.prepareStatement(badSql,…

… null) (idempiere#2184)

* IDEMPIERE-5994 Connection leak when using DB.prepareStatement(badSql, null)

* - implement approach suggested by Heng Sin

org.adempiere.base/src/org/compiere/db/PreparedStatementProxy.java

@@ -15,6 +15,7 @@
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
+import java.sql.SQLException;
 import java.util.logging.Level;
 
 import javax.sql.RowSet;
@@ -95,6 +96,11 @@ protected void init() {
 			p_stmt = conn.prepareStatement(p_vo.getSql(), p_vo
 					.getResultSetType(), p_vo.getResultSetConcurrency());
 		} catch (Exception e) {
+			try {
+				this.close();
+			} catch (SQLException e1) {
+				// ignore
+			}
 			log.log(Level.SEVERE, p_vo.getSql(), e);
 			throw new DBException(e);
 		}

org.adempiere.base/src/org/compiere/db/StatementProxy.java

@@ -184,7 +184,7 @@ protected void init()
 	 * @throws SQLException
 	 * @see java.sql.Statement#close()
 	 */
-	private void close () throws SQLException
+	protected void close () throws SQLException
 	{
 		if (close) return;