Added function to check syntax of URL for DNS lookup

For now, simply checks for '.' character, but that will be easy to
change in the future if necessary/desired.

2 files changed, 44 insertions, 8 deletions

diff --git a/src/common/dns_utils.cpp b/src/common/dns_utils.cpp
index 6755a30b2..60f00916c 100644
--- a/src/common/dns_utils.cpp
+++ b/src/common/dns_utils.cpp
@@ -79,11 +79,16 @@ DNSResolver::~DNSResolver()
 
 std::vector<std::string> DNSResolver::get_ipv4(const std::string& url)
 {
+  std::vector<std::string> addresses;
+
+  if (!check_address_syntax(url))
+  {
+    return addresses;
+  }
+
   // destructor takes care of cleanup
   ub_result_ptr result;
 
-  std::vector<std::string> retval;
-
   // call DNS resolver, blocking.  if return value not zero, something went wrong
   if (!ub_resolve(m_data->m_ub_context, url.c_str(), LDNS_RR_TYPE_A, LDNS_RR_CLASS_IN, &(result.ptr)))
   {
@@ -96,19 +101,25 @@ std::vector<std::string> DNSResolver::get_ipv4(const std::string& url)
         // convert bytes to string, append if no error
         if (inet_ntop(AF_INET, result.ptr->data[i], as_str, sizeof(as_str)))
         {
-          retval.push_back(as_str);
+          addresses.push_back(as_str);
         }
       }
     }
   }
 
-  return retval;
+  return addresses;
 }
 
 std::vector<std::string> DNSResolver::get_ipv6(const std::string& url)
 {
+  std::vector<std::string> addresses;
+
+  if (!check_address_syntax(url))
+  {
+    return addresses;
+  }
+
   ub_result_ptr result;
-  std::vector<std::string> retval;
 
   // call DNS resolver, blocking.  if return value not zero, something went wrong
   if (!ub_resolve(m_data->m_ub_context, url.c_str(), LDNS_RR_TYPE_AAAA, LDNS_RR_CLASS_IN, &(result.ptr)))
@@ -122,20 +133,26 @@ std::vector<std::string> DNSResolver::get_ipv6(const std::string& url)
         // convert bytes to string, append if no error
         if (inet_ntop(AF_INET6, result.ptr->data[i], as_str, sizeof(as_str)))
         {
-          retval.push_back(as_str);
+          addresses.push_back(as_str);
         }
       }
     }
   }
 
-  return retval;
+  return addresses;
 }
 
 std::vector<std::string> DNSResolver::get_txt_record(const std::string& url)
 {
-  ub_result_ptr result;
   std::vector<std::string> records;
 
+  if (!check_address_syntax(url))
+  {
+    return records;
+  }
+
+  ub_result_ptr result;
+
   // call DNS resolver, blocking.  if return value not zero, something went wrong
   if (!ub_resolve(m_data->m_ub_context, url.c_str(), LDNS_RR_TYPE_TXT, LDNS_RR_CLASS_IN, &(result.ptr)))
   {
@@ -161,4 +178,14 @@ DNSResolver& DNSResolver::instance()
   return *staticInstance;
 }
 
+bool DNSResolver::check_address_syntax(const std::string& addr)
+{
+  // if string doesn't contain a dot, we won't consider it a url for now.
+  if (addr.find(".") == std::string::npos)
+  {
+    return false;
+  }
+  return true;
+}
+
 }  // namespace tools
diff --git a/src/common/dns_utils.h b/src/common/dns_utils.h
index e57e55861..375f6ba30 100644
--- a/src/common/dns_utils.h
+++ b/src/common/dns_utils.h
@@ -101,6 +101,15 @@ public:
 
 private:
 
+  /**
+   * @brief Checks a string to see if it looks like a URL
+   *
+   * @param addr the string to be checked
+   *
+   * @return true if it looks enough like a URL, false if not
+   */
+  bool check_address_syntax(const std::string& addr);
+
   DNSResolverData *m_data;
 }; // class DNSResolver