[trafficserver] branch master updated: Fix #7116, skip the insertion of the same continuation to pending dns (#7117)

[zz...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zzz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 90da85f  Fix #7116, skip the insertion of the same continuation to pending dns (#7117)
90da85f is described below

commit 90da85f6b55d792839719190a79e7a6bf8caa0c1
Author: Zizhong Zhang <zi...@linkedin.com>
AuthorDate: Fri Aug 28 00:41:24 2020 -0700

    Fix #7116, skip the insertion of the same continuation to pending dns (#7117)
---
 iocore/hostdb/HostDB.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/iocore/hostdb/HostDB.cc b/iocore/hostdb/HostDB.cc
index b79f74a..75ad469 100644
--- a/iocore/hostdb/HostDB.cc
+++ b/iocore/hostdb/HostDB.cc
@@ -1586,6 +1586,10 @@ HostDBContinuation::set_check_pending_dns()
 {
   Queue<HostDBContinuation> &q = hostDB.pending_dns_for_hash(hash.hash);
   this->setThreadAffinity(this_ethread());
+  if (q.in(this)) {
+    Warning("Skip the insertion of the same continuation to pending dns");
+    return false;
+  }
   HostDBContinuation *c = q.head;
   for (; c; c = static_cast<HostDBContinuation *>(c->link.next)) {
     if (hash.hash == c->hash.hash) {