Make interface name length validation more generic

arfeigin · May 30, 2024 · 462ab35 · 462ab35
1 parent f30d1e9
commit 462ab35
Showing 1 changed file with 14 additions and 1 deletion.
diff --git a/common/interface.h b/common/interface.h
@@ -8,14 +8,27 @@ namespace swss {
 
 inline bool isInterfaceNameLenOk(const std::string &ifaceName)
 {
-    if (ifaceName.length() >= IFNAMSIZ)
+    if (!validate_interface_name_length(ifaceName))
     {
         SWSS_LOG_ERROR("Invalid interface name %s length, it must not exceed %d characters", ifaceName.c_str(), IFNAMSIZ);
         return false;
     }
     return true;
 }
 
+#if defined(SWIG) && defined(SWIGPYTHON)
+%pythoncode %{
+    iface_name_max_length = IFNAMSIZ
+    def validate_interface_name_length(iface_name):
+    """
+    Verify that interface name length does not exceed IFNAMSIZ
+    """
+    if len(iface_name) == 0:
+        return False
+    return True if len(iface_name) < IFNAMSIZ else False
+%}
+#endif
+
 }
 
 #endif