From: Bert Hubert <bert.hubert@netherlabs.nl>
Date: Thu, 2 Mar 2006 19:32:49 +0000 (+0000)
Subject: remove some duplicate ;'s
X-Git-Tag: pdns-2.9.20~23
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=5762f14e3c7c9c5a37e4dfb94078a219fdffc51a;p=thirdparty%2Fpdns.git

remove some duplicate ;'s
teach dynhandler to reject non-ip address parameters to notify-host
plus defense in depth, make powerdns not exit if asked to notify non-ip address
Close ticket 24


git-svn-id: svn://svn.powerdns.com/pdns/trunk/pdns@565 d19b8d6e-7fed-0310-83ef-9ca221ded41b
---

diff --git a/pdns/communicator.cc b/pdns/communicator.cc
index 48b6c74b42..42021ddb9b 100644
--- a/pdns/communicator.cc
+++ b/pdns/communicator.cc
@@ -317,8 +317,13 @@ int CommunicatorClass::doNotifications()
   bool purged;
   while(d_nq.getOne(domain, ip, &id, purged)) {
     if(!purged) {
-      d_nresolver.notify(d_nsock,domain,ip,id);
-      drillHole(domain,ip);
+      try {
+	d_nresolver.notify(d_nsock, domain, ip, id);
+	drillHole(domain, ip);
+      }
+      catch(ResolverException &re) {
+	L<<Logger::Error<<"Error trying to resolve '"+ip+"' for notifying '"+domain+"' to server: "+re.reason<<endl;
+      }
     }
     else
       L<<Logger::Error<<Logger::NTLog<<"Notification for "<<domain<<" to "<<ip<<" failed after retries"<<endl;
@@ -393,7 +398,7 @@ void CommunicatorClass::makeNotifySocket()
 
 void CommunicatorClass::notify(const string &domain, const string &ip)
 {
-  d_nq.add(domain,ip);
+  d_nq.add(domain, ip);
 
   d_any_sem.post();
 }
diff --git a/pdns/dnspacket.cc b/pdns/dnspacket.cc
index ffff05f07a..3ec0499dd0 100644
--- a/pdns/dnspacket.cc
+++ b/pdns/dnspacket.cc
@@ -433,9 +433,9 @@ void DNSPacket::fillSOAData(const string &content, SOAData &data)
   // fill out data with some plausible defaults:
   // 10800 3600 604800 3600
   data.serial=0;
-  data.refresh=arg().asNum("soa-refresh-default");;
-  data.retry=arg().asNum("soa-retry-default");;
-  data.expire=arg().asNum("soa-expire-default");;
+  data.refresh=arg().asNum("soa-refresh-default");
+  data.retry=arg().asNum("soa-retry-default");
+  data.expire=arg().asNum("soa-expire-default");
   data.default_ttl=arg().asNum("soa-minimum-ttl");
 
   vector<string>parts;
diff --git a/pdns/dynhandler.cc b/pdns/dynhandler.cc
index bba95a7d34..3bdb45c513 100644
--- a/pdns/dynhandler.cc
+++ b/pdns/dynhandler.cc
@@ -203,6 +203,11 @@ string DLNotifyHostHandler(const vector<string>&parts, Utility::pid_t ppid)
   ostringstream os;
   if(parts.size()!=3)
     return "syntax: notify-host domain ip";
+
+  struct in_addr inp;
+  if(!Utility::inet_aton(parts[2].c_str(),&inp))
+    return "Unable to convert '"+parts[2]+"' to an IP address";
+
   L<<Logger::Warning<<"Notification request to host "<<parts[2]<<" for domain '"<<parts[1]<<"' received"<<endl;
   Communicator.notify(parts[1],parts[2]);
   return "Added to queue";
diff --git a/pdns/resolver.cc b/pdns/resolver.cc
index d95c2c4dfc..db18111a1c 100644
--- a/pdns/resolver.cc
+++ b/pdns/resolver.cc
@@ -138,7 +138,8 @@ int Resolver::notify(int sock, const string &domain, const string &ip, uint16_t
   p.spoofID(id);
 
   struct in_addr inp;
-  Utility::inet_aton(ip.c_str(),&inp);
+  if(!Utility::inet_aton(ip.c_str(),&inp))
+    throw ResolverException("Unable to convert '"+ip+"' to an IP address");
 
   struct sockaddr_in toaddr;
   toaddr.sin_addr.s_addr=inp.s_addr;