You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@juddi.apache.org by ks...@apache.org on 2008/12/05 05:23:05 UTC
svn commit: r723580 -
/webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java
Author: kstam
Date: Thu Dec 4 20:23:05 2008
New Revision: 723580
URL: http://svn.apache.org/viewvc?rev=723580&view=rev
Log:
JUDDI-128 fixing connection leak
Modified:
webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java
Modified: webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java
URL: http://svn.apache.org/viewvc/webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java?rev=723580&r1=723579&r2=723580&view=diff
==============================================================================
--- webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java (original)
+++ webservices/juddi/trunk/src/main/java/org/apache/juddi/registry/RegistryEngine.java Thu Dec 4 20:23:05 2008
@@ -245,8 +245,9 @@
String tablePrefix = Config.getStringProperty(
RegistryEngine.PROPNAME_TABLE_PREFIX,RegistryEngine.DEFAULT_TABLE_PREFIX);
+ Connection conn=null;
try {
- Connection conn = ConnectionManager.acquireConnection();
+ conn = ConnectionManager.acquireConnection();
boolean create = false;
Statement st = conn.createStatement();
@@ -270,6 +271,12 @@
}
} catch (Exception e) {
log.error("Could not create jUDDI database " + e.getMessage(), e);
+ } finally {
+ if (conn!=null) {
+ try {
+ if (!conn.isClosed()) conn.close();
+ } catch (SQLException e) {}
+ }
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: juddi-cvs-unsubscribe@ws.apache.org
For additional commands, e-mail: juddi-cvs-help@ws.apache.org