From: bert hubert <bert.hubert@netherlabs.nl>
Date: Wed, 1 Apr 2015 11:28:07 +0000 (+0200)
Subject: add error checking on server generation, improving the error message
X-Git-Tag: dnsdist-1.0.0-alpha1~248^2~88^2~19^2~1
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=cbbd15333c7a2ccde6e825dd423042fe7e142fd4;p=pdns

add error checking on server generation, improving the error message
---

diff --git a/pdns/dnsdist-lua.cc b/pdns/dnsdist-lua.cc
index b83f6e1db..853d47379 100644
--- a/pdns/dnsdist-lua.cc
+++ b/pdns/dnsdist-lua.cc
@@ -22,7 +22,15 @@ vector<std::function<void(void)>> setupLua(bool client, const std::string& confi
 			  return std::make_shared<DownstreamState>(ComboAddress());
 			}
 			if(auto address = boost::get<string>(&pvars)) {
-			  auto ret=std::make_shared<DownstreamState>(ComboAddress(*address, 53));
+			  std::shared_ptr<DownstreamState> ret;
+			  try {
+			    ret=std::make_shared<DownstreamState>(ComboAddress(*address, 53));
+			  }
+			  catch(std::exception& e) {
+			    g_outputBuffer="Error creating new server: "+string(e.what());
+			    errlog("Error creating new server with address %s: %s", *address, e.what());
+			    return ret;
+			  }
 
 			  if(qps) {
 			    ret->qps=QPSLimiter(*qps, *qps);
@@ -47,7 +55,15 @@ vector<std::function<void(void)>> setupLua(bool client, const std::string& confi
 			  return ret;
 			}
 			auto vars=boost::get<newserver_t>(pvars);
-			auto ret=std::make_shared<DownstreamState>(ComboAddress(boost::get<string>(vars["address"]), 53));
+			std::shared_ptr<DownstreamState> ret;
+			try {
+			  ret=std::make_shared<DownstreamState>(ComboAddress(boost::get<string>(vars["address"]), 53));
+			}
+			catch(std::exception& e) {
+			  g_outputBuffer="Error creating new server: "+string(e.what());
+			  errlog("Error creating new server with address %s: %s", boost::get<string>(vars["address"]), e.what());
+			  return ret;
+			}
 			
 			if(vars.count("qps")) {
 			  int qps=boost::lexical_cast<int>(boost::get<string>(vars["qps"]));