]> granicus.if.org Git - pdns/commitdiff
Fixed few nits
authorAki Tuomi <cmouse@desteem.org>
Thu, 14 Nov 2013 19:43:34 +0000 (21:43 +0200)
committerAki Tuomi <cmouse@desteem.org>
Mon, 18 Nov 2013 17:44:29 +0000 (19:44 +0200)
pdns/resolver.cc
regression-tests/check_stest_source
regression-tests/start-test-stop

index fbf9fae5f8c8f54a617e34da2c153bdd793f7335..582ad7b0d4b67f86ef0d8d63a46ecc00048f7f24 100644 (file)
@@ -102,9 +102,10 @@ catch(...) {
 
 Resolver::~Resolver()
 {
-   for(std::map<std::string,int>::iterator iter = locals.begin(); iter != locals.end(); iter++) {
-       close(iter->second);
-   }
+  for(std::map<std::string,int>::iterator iter = locals.begin(); iter != locals.end(); iter++) {
+    if (iter->second >= 0)
+     close(iter->second);
+  }
 }
 
 uint16_t Resolver::sendResolve(const ComboAddress& remote, const ComboAddress& local,
@@ -152,6 +153,7 @@ uint16_t Resolver::sendResolve(const ComboAddress& remote, const ComboAddress& l
      } else {
         // try to make socket
         sock = makeQuerySocket(local, true); 
+        Utility::setNonBlocking( sock );
         locals[lstr] = sock;
      }
   }
index b7d3bf1a6e398f1222a5c6bf83b9669bd53f83f5..57aa58dc28f88b550fec87d0d334da04bc1ecaab 100644 (file)
@@ -1,7 +1,9 @@
 function prequery ( dnspacket )
        qname, qtype = dnspacket:getQuestion()
         remote = dnspacket:getRemote()
-       if qname == "stest.com" and remote != "127.0.0.2":
-               return false
-       return true
+       if qname == "stest.com" and remote ~= "127.0.0.2" then
+                dnspacket:setRcode(pdns.NXDOMAIN)
+               return true
+        end
+       return false
 end
index f146523cd261b6beb12f0f58b2b0d7d6151f67f1..3938a659be5595f8c9b46d5fe6abd1e417e2e4f3 100755 (executable)
@@ -929,7 +929,7 @@ EOF
 esac
 
 if [ "$skiplua" == "1" ]; then
-  skipreasons="$skipreaasons nolua"
+  skipreasons="$skipreasons nolua"
 fi
 
 check_process