From: bert hubert <bert.hubert@netherlabs.nl>
Date: Wed, 19 Dec 2018 09:08:39 +0000 (+0100)
Subject: prevent leak of file descriptor if running out of ports for incoming AXFR
X-Git-Tag: rec-4.2.0-alpha1~80^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F7294%2Fhead;p=thirdparty%2Fpdns.git

prevent leak of file descriptor if running out of ports for incoming AXFR
---

diff --git a/pdns/resolver.cc b/pdns/resolver.cc
index ab3dcad0ce..fb8c830dbd 100644
--- a/pdns/resolver.cc
+++ b/pdns/resolver.cc
@@ -86,10 +86,11 @@ int makeQuerySocket(const ComboAddress& local, bool udpOrTCP, bool nonLocalBind)
   }
   else {
     // tcp, let the kernel figure out the port
-    // cerr<<"letting kernel pick TCP port"<<endl;
     ourLocal.sin4.sin_port = 0;
-    if(::bind(sock, (struct sockaddr *)&ourLocal, ourLocal.getSocklen()) < 0)
+    if(::bind(sock, (struct sockaddr *)&ourLocal, ourLocal.getSocklen()) < 0) {
+      closesocket(sock);
       throw PDNSException("Resolver binding to local TCP socket on "+ourLocal.toString()+": "+stringerror());
+    }
   }
   return sock;
 }