Try to be abit more smart in recognizing IP:port combos for both

IPV4 and IPv6.

Should fix #7743

diff --git a/pdns/reczones.cc b/pdns/reczones.cc
index 87ec45732c1014a4994feb20c861e8eacca4048b..487781b2893f580d6d455a421b0332a415318b08 100644 (file)
--- a/pdns/reczones.cc
+++ b/pdns/reczones.cc
@@ -261,20 +261,24 @@ static void makeIPToNamesZone(std::shared_ptr<SyncRes::domainmap_t> newMap, cons
 
 ComboAddress parseIPAndPort(const std::string& input, uint16_t port)
 {
-  if(input.find(':') == string::npos || input.empty()) // common case
+  if (input.find(':') == string::npos || input.empty()) { // common case
     return ComboAddress(input, port);
+  }
 
   pair<string,string> both;
-
   try { // case 2
-    both=splitField(input,':');
-    uint16_t newport=static_cast<uint16_t>(pdns_stou(both.second));
+    string::size_type cpos = input.rfind(':');
+    both.first = input.substr(0, cpos);
+    both.second = input.substr(cpos + 1);
+
+    uint16_t newport = static_cast<uint16_t>(pdns_stou(both.second));
     return ComboAddress(both.first, newport);
-  } 
-  catch(...){}
+  }
+  catch(...) {
+  }
 
-  if(input[0]=='[') { // case 4
-    both=splitField(input.substr(1),']');
+  if (input[0] == '[') { // case 4
+    both = splitField(input.substr(1), ']');
     return ComboAddress(both.first, both.second.empty() ? port : static_cast<uint16_t>(pdns_stou(both.second.substr(1))));
   }