rec: release ref to key in MTasker::waitEvent

If we do not do that, only the next call of sendEvent will release
the shared pointer to the TCPConnectionHanlder in PacketID. This
might delay cleaning TCP objects up on not-so-busy threads.

Fixes #13422, thought it can still take a while on idle recursors,
as the housekeeping function is not called often in those cases.

diff --git a/pdns/recursordist/mtasker.hh b/pdns/recursordist/mtasker.hh
index a45f64d8326b75d6627bd5d9612a731fd8b64532..1287e94455e4bb68e90a80cb085fe43eb429c62c 100644 (file)
--- a/pdns/recursordist/mtasker.hh
+++ b/pdns/recursordist/mtasker.hh
@@ -305,7 +305,7 @@ int MTasker<EventKey, EventVal, Cmp>::waitEvent(EventKey& key, EventVal* val, un
   if ((char*)&waiter < d_threads[d_tid].highestStackSeen) {
     d_threads[d_tid].highestStackSeen = (char*)&waiter;
   }
-  key = d_eventkey;
+  key = std::move(d_eventkey);
   return d_waitstatus;
 }