summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog5
-rw-r--r--ext/socket/lib/socket.rb1
-rw-r--r--test/socket/test_unix.rb3
3 files changed, 9 insertions, 0 deletions
diff --git a/ChangeLog b/ChangeLog
index 4af8b7d3a1..df1fb46aaa 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Wed Feb 11 16:54:26 2009 Tanaka Akira <akr@fsij.org>
+
+ * ext/socket/lib/socket.rb (Socket.unix_server_socket): close the
+ socket when the block exits.
+
Wed Feb 11 16:50:59 2009 Tanaka Akira <akr@fsij.org>
* ext/socket/lib/socket.rb (Socket.unix_server_socket): call the block
diff --git a/ext/socket/lib/socket.rb b/ext/socket/lib/socket.rb
index 1099b7fd2f..adc28147b3 100644
--- a/ext/socket/lib/socket.rb
+++ b/ext/socket/lib/socket.rb
@@ -612,6 +612,7 @@ class Socket
begin
yield s
ensure
+ s.close if !s.closed?
File.unlink path
end
else
diff --git a/test/socket/test_unix.rb b/test/socket/test_unix.rb
index 2b1f4deb33..6e822dbc36 100644
--- a/test/socket/test_unix.rb
+++ b/test/socket/test_unix.rb
@@ -299,10 +299,13 @@ class TestSocket_UNIXSocket < Test::Unit::TestCase
def test_unix_server_socket
Dir.mktmpdir {|d|
path = "#{d}/sock"
+ s0 = nil
Socket.unix_server_socket(path) {|s|
assert_equal(path, s.local_address.unix_path)
assert(File.socket?(path))
+ s0 = s
}
+ assert(s0.closed?)
assert_raise(Errno::ENOENT) { File.stat path }
}
end