aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKim Alvefur <zash@zash.se>2014-06-30 12:45:53 +0200
committerKim Alvefur <zash@zash.se>2014-06-30 12:45:53 +0200
commitd2e295445184a7fed98f158968e509b1258d3868 (patch)
tree7e6304e340020f8114d8707e07f3af5ee8a476ba
parentca8f685253b29e92d052d8c05d0398f02fccc318 (diff)
downloadprosody-d2e295445184a7fed98f158968e509b1258d3868.tar.gz
prosody-d2e295445184a7fed98f158968e509b1258d3868.zip
net.dns: Fix duplicated cache insertions by limiting outstanding queries per name to one
-rw-r--r--net/dns.lua9
1 files changed, 8 insertions, 1 deletions
diff --git a/net/dns.lua b/net/dns.lua
index cdd949a2..492bd1c5 100644
--- a/net/dns.lua
+++ b/net/dns.lua
@@ -715,6 +715,14 @@ end
function resolver:query(qname, qtype, qclass) -- - - - - - - - - - -- query
qname, qtype, qclass = standardize(qname, qtype, qclass)
+ local co = coroutine.running();
+ local q = get(self.wanted, qclass, qtype, qname);
+ if co and q then
+ -- We are already waiting for a reply to an identical query.
+ set(self.wanted, qclass, qtype, qname, co, true);
+ return true;
+ end
+
if not self.server then self:adddefaultnameservers(); end
local question = encodeQuestion(qname, qtype, qclass);
@@ -735,7 +743,6 @@ function resolver:query(qname, qtype, qclass) -- - - - - - - - - - -- query
self.active[id][question] = o;
-- remember which coroutine wants the answer
- local co = coroutine.running();
if co then
set(self.wanted, qclass, qtype, qname, co, true);
--set(self.yielded, co, qclass, qtype, qname, true);