diff options
author | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-05-29 00:06:17 +0000 |
---|---|---|
committer | akr <akr@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-05-29 00:06:17 +0000 |
commit | 089d2ddda44b5c7f7e1e6c69103b6f47716786d2 (patch) | |
tree | 0d62d53f9ccc0b760bb3a2291ef5c024ebff464d | |
parent | 2dce743725236e0bdfd99ed005bc6266a2c943b4 (diff) | |
download | ruby-089d2ddda44b5c7f7e1e6c69103b6f47716786d2.tar.gz |
* lib/resolv.rb (Resolv::DNS::Requester#request): rescue ECONNREFUSED.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28058 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | lib/resolv.rb | 8 | ||||
-rw-r--r-- | test/resolv/test_dns.rb | 14 |
3 files changed, 25 insertions, 1 deletions
@@ -1,3 +1,7 @@ +Sat May 29 09:04:06 2010 Tanaka Akira <akr@fsij.org> + + * lib/resolv.rb (Resolv::DNS::Requester#request): rescue ECONNREFUSED. + Sat May 29 08:46:29 2010 Nobuyoshi Nakada <nobu@ruby-lang.org> * gc.c (allocate_sorted_heaps, before_gc_sweep, gc_sweep): removed diff --git a/lib/resolv.rb b/lib/resolv.rb index 9a96c55931..f238439116 100644 --- a/lib/resolv.rb +++ b/lib/resolv.rb @@ -638,7 +638,13 @@ class Resolv if !select_result raise ResolvTimeout end - reply, from = recv_reply(select_result[0]) + begin + reply, from = recv_reply(select_result[0]) + rescue Errno::ECONNREFUSED + # No name server running on the server? + # Don't wait anymore. + raise ResolvTimeout + end begin msg = Message.decode(reply) rescue DecodeError diff --git a/test/resolv/test_dns.rb b/test/resolv/test_dns.rb index 0ab59665a3..9a9c33ea84 100644 --- a/test/resolv/test_dns.rb +++ b/test/resolv/test_dns.rb @@ -104,4 +104,18 @@ class TestResolvDNS < Test::Unit::TestCase end } end + + def test_no_server + u = UDPSocket.new + u.bind("127.0.0.1", 0) + _, port, _, host = u.addr + u.close + # A rase condition here. + # Another program may use the port. + # But no way to prevent it. + Resolv::DNS.open(:nameserver_port => [[host, port]]) {|dns| + assert_equal([], dns.getresources("test-no-server.example.org", Resolv::DNS::Resource::IN::A)) + } + end + end |