Add helper function to validate interface name length (#931)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
Co-authored-by: afeigin <afeigin@nvidia.com>

common/Makefile.am

@@ -69,6 +69,7 @@ common_libswsscommon_la_SOURCES = \
     common/zmqserver.cpp             \
     common/asyncdbupdater.cpp        \
     common/redis_table_waiter.cpp    \
+    common/interface.h               \
     common/c-api/util.cpp                  \
     common/c-api/dbconnector.cpp           \
     common/c-api/consumerstatetable.cpp    \

common/interface.h

@@ -0,0 +1,19 @@
+#ifndef __INTERFACE__
+#define __INTERFACE__
+
+#include <string>
+#include <net/if.h>
+
+namespace swss
+{
+
+const size_t IFACE_NAME_MAX_LEN = IFNAMSIZ - 1;
+
+bool isInterfaceNameValid(const std::string &ifaceName)
+{
+    return !ifaceName.empty() && (ifaceName.length() < IFNAMSIZ);
+}
+
+}
+
+#endif

pyext/swsscommon.i

@@ -58,6 +58,7 @@
 #include "zmqproducerstatetable.h"
 #include <memory>
 #include <functional>
+#include "interface.h"
 %}
 
 %include <std_string.i>
@@ -282,6 +283,7 @@ T castSelectableObj(swss::Selectable *temp)
 %include "zmqserver.h"
 %include "zmqclient.h"
 %include "zmqconsumerstatetable.h"
+%include "interface.h"
 
 %extend swss::DBConnector {
     %template(hgetall) hgetall<std::map<std::string, std::string>>;
@@ -296,7 +298,7 @@ T castSelectableObj(swss::Selectable *temp)
 %include "table.h"
 #ifdef ENABLE_YANG_MODULES
 %include "decoratortable.h"
-#endif 
+#endif
 %clear std::vector<std::string> &keys;
 %clear std::vector<std::string> &ops;
 %clear std::vector<std::vector<std::pair<std::string, std::string>>> &fvss;

tests/test_interface.py

@@ -0,0 +1,8 @@
+from swsscommon import swsscommon
+
+def test_is_interface_name_valid():
+    invalid_interface_name = "TooLongInterfaceName"
+    assert not swsscommon.isInterfaceNameValid(invalid_interface_name)
+
+    validInterfaceName = "OkInterfaceName"
+    assert swsscommon.isInterfaceNameValid(validInterfaceName)