commit ff12acbb8c02d965232e14829518bc6c4d34e53d
author William A. Kennington III <wak@google.com> Fri Oct 07 19:06:56 2022 -0700
committer William A. Kennington III <wak@google.com> Tue Oct 25 03:07:00 2022 -0700
tree 18e2e7634edd7de21d4389c266b4ab0a4c6935d8
parent d07c08628a19e505f45a6bd4338eb6bac597b772

util: Add non-family isValidIP

Change-Id: Ib039658d466c0b10b6bc3d1dde9bf5926a124c65
Signed-off-by: William A. Kennington III <wak@google.com>

src/ethernet_interface.cpp

diff --git a/src/ethernet_interface.cpp b/src/ethernet_interface.cpp
index 87c660d..e9f658a 100644
--- a/src/ethernet_interface.cpp
+++ b/src/ethernet_interface.cpp
@@ -297,7 +297,7 @@
 ObjectPath EthernetInterface::neighbor(std::string ipAddress,
                                        std::string macAddress)
 {
-    if (!isValidIP(AF_INET, ipAddress) && !isValidIP(AF_INET6, ipAddress))
+    if (!isValidIP(ipAddress))
     {
         log<level::ERR>("Not a valid IP address",
                         entry("ADDRESS=%s", ipAddress.c_str()));
@@ -773,8 +773,7 @@
 {
     for (const auto& nameserverip : value)
     {
-        if (!isValidIP(AF_INET, nameserverip) &&
-            !isValidIP(AF_INET6, nameserverip))
+        if (!isValidIP(nameserverip))
         {
             log<level::ERR>("Not a valid IP address"),
                 entry("ADDRESS=%s", nameserverip.c_str());

src/util.cpp

diff --git a/src/util.cpp b/src/util.cpp
index 7ede99c..b1f4d39 100644
--- a/src/util.cpp
+++ b/src/util.cpp
@@ -189,6 +189,11 @@
     return inet_pton(family, address.c_str(), buf) > 0;
 }
 
+bool isValidIP(stdplus::const_zstring address) noexcept
+{
+    return isValidIP(AF_INET, address) || isValidIP(AF_INET6, address);
+}
+
 bool isValidPrefix(int family, uint8_t prefix)
 {
     return familyVisit(

src/util.hpp

diff --git a/src/util.hpp b/src/util.hpp
index 52abfb7..16dca64 100644
--- a/src/util.hpp
+++ b/src/util.hpp
@@ -131,6 +131,7 @@
  * @returns true if it is valid otherwise false.
  */
 bool isValidIP(int family, stdplus::const_zstring address) noexcept;
+bool isValidIP(stdplus::const_zstring address) noexcept;
 
 /* @brief checks that the given prefix is valid or not.
  * @param[in] family - IP address family(AF_INET/AF_INET6).