dnsdist: Fix the accounting of 'active' connections

diff --git a/pdns/dnsdistdist/dnsdist-tcp-downstream.cc b/pdns/dnsdistdist/dnsdist-tcp-downstream.cc
index 03a4c41b244acb4812ad7b686782a1f85ee838b9..0a5a11b1df7a1c2c53110f6c7bffeb47ed008814 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-tcp-downstream.cc
+++ b/pdns/dnsdistdist/dnsdist-tcp-downstream.cc
@@ -440,9 +440,9 @@ IOState TCPConnectionToBackend::handleResponse(std::shared_ptr<TCPConnectionToBa
       return IOState::Done;
     }
     auto ids = std::move(it->second.d_idstate);
+    d_pendingResponses.erase(it);
     DEBUGLOG("passing response to client connection for "<<ids.qname);
     clientConn->handleResponse(clientConn, now, TCPResponse(std::move(d_responseBuffer), std::move(ids), conn));
-    d_pendingResponses.erase(it);
 
     if (!d_pendingQueries.empty()) {
       DEBUGLOG("still have some queries to send");