From 7934d276c1dfdc3224c391c23f574d244c5b0a10 Mon Sep 17 00:00:00 2001
From: Matthew Swift <matthew.swift@forgerock.com>
Date: Thu, 29 Nov 2012 23:42:35 +0000
Subject: [PATCH] Minor code cleanup.

---
 opendj3/opendj-ldap-sdk/src/test/java/com/forgerock/opendj/ldap/DefaultTCPNIOTransportTestCase.java |   13 +++++++++----
 1 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/opendj3/opendj-ldap-sdk/src/test/java/com/forgerock/opendj/ldap/DefaultTCPNIOTransportTestCase.java b/opendj3/opendj-ldap-sdk/src/test/java/com/forgerock/opendj/ldap/DefaultTCPNIOTransportTestCase.java
index 1e3c5bd..7058267 100644
--- a/opendj3/opendj-ldap-sdk/src/test/java/com/forgerock/opendj/ldap/DefaultTCPNIOTransportTestCase.java
+++ b/opendj3/opendj-ldap-sdk/src/test/java/com/forgerock/opendj/ldap/DefaultTCPNIOTransportTestCase.java
@@ -22,6 +22,7 @@
  *
  *
  *      Copyright 2010 Sun Microsystems, Inc.
+ *      Portions copyright 2012 ForgeRock AS.
  */
 
 package com.forgerock.opendj.ldap;
@@ -57,10 +58,14 @@
 
         // Establish a socket connection to see if the transport factory works.
         final Socket socket = new Socket();
-        socket.connect(socketAddress);
+        try {
+            socket.connect(socketAddress);
 
-        // Successfully connected if there is no exception.
-        assertTrue(socket.isConnected());
-        // Don't stop the transport because it is shared with the ldap server.
+            // Successfully connected if there is no exception.
+            assertTrue(socket.isConnected());
+            // Don't stop the transport because it is shared with the ldap server.
+        } finally {
+            socket.close();
+        }
     }
 }

--
Gitblit v1.10.0