Fix review comments

diff --git a/pdns/dnswriter.cc b/pdns/dnswriter.cc
index 4f7656f60034e2c8a39df0127e2b87b230547bde..d788217e8c7cb4c5ab89bc19d3af702d18a79813 100644 (file)
--- a/pdns/dnswriter.cc
+++ b/pdns/dnswriter.cc
@@ -110,7 +110,7 @@ void DNSPacketWriter::addOpt(uint16_t udpsize, int extRCode, int Z, const vector
    *        indicates that an unextended RCODE is in use (values 0 through 15).
    */
   stuff.extRCode = extRCode>>4;
-  if (extRCode > 15) { // As this trumps the existing RCODE
+  if (extRCode != 0) { // As this trumps the existing RCODE
     getHeader()->rcode = extRCode;
   }
 

diff --git a/pdns/pdns_recursor.cc b/pdns/pdns_recursor.cc
index beaccd2f9ebdcc2e97a0eaf3502c2c0c79a1f765..eb481d2f79bb42645c79470c79d38807272975e0 100644 (file)
--- a/pdns/pdns_recursor.cc
+++ b/pdns/pdns_recursor.cc
@@ -3195,6 +3195,11 @@ static int serviceMain(int argc, char*argv[])
     g_quiet=false;
     g_dnssecLOG=true;
   }
+  char myHostname[MAXHOSTNAMELEN];
+  if (gethostname(myHostname, sizeof(myHostname)-1) == 0){
+  } else {
+    g_log<<Logger::Warning<<"Unable to get the hostname, NSID and id.server values will be empty: "<<strerror(errno)<<endl;
+  }
 
   SyncRes::s_minimumTTL = ::arg().asNum("minimum-ttl-override");
 
@@ -3214,12 +3219,7 @@ static int serviceMain(int argc, char*argv[])
   SyncRes::s_maxdepth=::arg().asNum("max-recursion-depth");
   SyncRes::s_rootNXTrust = ::arg().mustDo( "root-nx-trust");
   if(SyncRes::s_serverID.empty()) {
-    char tmp[128];
-    if (gethostname(tmp, sizeof(tmp)-1) == 0){
-      SyncRes::s_serverID = tmp;
-    } else {
-      g_log<<Logger::Warning<<"Unable to get the hostname, NSID and id.server values will be empty: "<<strerror(errno)<<endl;
-    }
+    SyncRes::s_serverID = myHostname;
   }
 
   SyncRes::s_ecsipv4limit = ::arg().asNum("ecs-ipv4-bits");
@@ -3334,9 +3334,7 @@ static int serviceMain(int argc, char*argv[])
   dns_random_init();
 
   if(::arg()["server-id"].empty()) {
-    char tmp[128];
-    gethostname(tmp, sizeof(tmp)-1);
-    ::arg().set("server-id") = tmp;
+    ::arg().set("server-id") = myHostname;
   }
 
   int newgid=0;