[jboss-cvs] JBossAS SVN: r77848 - branches/Branch_4_2/transaction/src/main/org/jboss/tm.

[jboss-cvs-commits at lists.jboss.org]

Author: galder.zamarreno at jboss.com
Date: 2008-09-03 10:35:38 -0400 (Wed, 03 Sep 2008)
New Revision: 77848

Modified:
   branches/Branch_4_2/transaction/src/main/org/jboss/tm/TransactionPropagationContextUtil.java
Log:
[JBAS-5906] Close naming contexts on finally.

Modified: branches/Branch_4_2/transaction/src/main/org/jboss/tm/TransactionPropagationContextUtil.java
===================================================================
--- branches/Branch_4_2/transaction/src/main/org/jboss/tm/TransactionPropagationContextUtil.java	2008-09-03 14:34:51 UTC (rev 77847)
+++ branches/Branch_4_2/transaction/src/main/org/jboss/tm/TransactionPropagationContextUtil.java	2008-09-03 14:35:38 UTC (rev 77848)
@@ -29,6 +29,7 @@
  * Comment
  *
  * @author <a href="mailto:bill at jboss.org">Bill Burke</a>
+ * @author <a href="mailto:galder.zamarreno at jboss.com">Galder Zamarreno</a>
  * @version $Revision$
  */
 public class TransactionPropagationContextUtil
@@ -48,9 +49,16 @@
          try
          {
             InitialContext ctx = new InitialContext();
-            // Get the transaction propagation context factory
-            tpcFactory = (TransactionPropagationContextFactory)
-            ctx.lookup("java:/TransactionPropagationContextExporter");
+            try
+            {
+               // Get the transaction propagation context factory
+               tpcFactory = (TransactionPropagationContextFactory)
+               ctx.lookup("java:/TransactionPropagationContextExporter");
+            }
+            finally
+            {
+               ctx.close();
+            } 
          }
          catch (NamingException e)
          {
@@ -72,9 +80,16 @@
          try
          {
             InitialContext ctx = new InitialContext();
-            // and the transaction propagation context importer
-            tpcImporter = (TransactionPropagationContextImporter)
-            ctx.lookup("java:/TransactionPropagationContextImporter");
+            try
+            {
+               // and the transaction propagation context importer
+               tpcImporter = (TransactionPropagationContextImporter)
+               ctx.lookup("java:/TransactionPropagationContextImporter");
+            }
+            finally
+            {
+               ctx.close();
+            } 
          }
          catch (NamingException e)
          {