do not merge: cherry-picked 929b4855b8 from master branch

Change-Id: Ie20371234a531f65f523682a0d0c27394dc30afa
2010-06-09 14:27:43 -07:00
parent b00854d4c0
commit a2c6d5bf30
1 changed files with 6 additions and 1 deletions
--- a/services/java/com/android/server/WifiWatchdogService.java
+++ b/services/java/com/android/server/WifiWatchdogService.java
@@ -1217,8 +1217,9 @@ public class WifiWatchdogService {
        private static Random sRandom = new Random();
        
        static boolean isDnsReachable(int dns, int timeout) {
+            DatagramSocket socket = null;
            try {
-                DatagramSocket socket = new DatagramSocket();
+                socket = new DatagramSocket();
                
                // Set some socket properties
                socket.setSoTimeout(timeout);
@@ -1271,6 +1272,10 @@ public class WifiWatchdogService {
                    Slog.d(TAG, "DnsPinger.isReachable got an unknown exception", e);
                }
                return false;
+            } finally {
+                if (socket != null) {
+                    socket.close();
+                }
            }
        }