acl: API cleanup

[vpp.git] / extras / vom / vom / acl_list_cmds.cpp

diff --git a/extras/vom/vom/acl_list_cmds.cpp b/extras/vom/vom/acl_list_cmds.cpp
index 343b2e5..f873fd1 100644 (file)
--- a/extras/vom/vom/acl_list_cmds.cpp
+++ b/extras/vom/vom/acl_list_cmds.cpp
@@ -24,9 +24,10 @@ namespace list_cmds {
 static void
 to_vpp(const l2_rule& rule, vapi_type_macip_acl_rule& payload)
 {
-  payload.is_permit = rule.action().value();
-  rule.src_ip().to_vpp(&payload.is_ipv6, payload.src_ip_addr,
-                       &payload.src_ip_prefix_len);
+  payload.is_permit = (vapi_enum_acl_action)rule.action().value();
+  rule.src_ip().to_vpp((uint8_t*)&payload.src_prefix.address.af,
+                       (uint8_t*)&payload.src_prefix.address.un,
+                       &payload.src_prefix.len);
   rule.mac().to_bytes(payload.src_mac, 6);
   rule.mac_mask().to_bytes(payload.src_mac_mask, 6);
 }
@@ -34,13 +35,15 @@ to_vpp(const l2_rule& rule, vapi_type_macip_acl_rule& payload)
 static void
 to_vpp(const l3_rule& rule, vapi_type_acl_rule& payload)
 {
-  payload.is_permit = rule.action().value();
-  rule.src().to_vpp(&payload.is_ipv6, payload.src_ip_addr,
-                    &payload.src_ip_prefix_len);
-  rule.dst().to_vpp(&payload.is_ipv6, payload.dst_ip_addr,
-                    &payload.dst_ip_prefix_len);
-
-  payload.proto = rule.proto();
+  payload.is_permit = (vapi_enum_acl_action)rule.action().value();
+  rule.src().to_vpp((uint8_t*)&payload.src_prefix.address.af,
+                    (uint8_t*)&payload.src_prefix.address.un,
+                    &payload.src_prefix.len);
+  rule.dst().to_vpp((uint8_t*)&payload.dst_prefix.address.af,
+                    (uint8_t*)&payload.dst_prefix.address.un,
+                    &payload.dst_prefix.len);
+
+  payload.proto = (vapi_enum_ip_proto)rule.proto();
   payload.srcport_or_icmptype_first = rule.srcport_or_icmptype_first();
   payload.srcport_or_icmptype_last = rule.srcport_or_icmptype_last();
   payload.dstport_or_icmpcode_first = rule.dstport_or_icmpcode_first();
@@ -50,7 +53,7 @@ to_vpp(const l3_rule& rule, vapi_type_acl_rule& payload)
   payload.tcp_flags_value = rule.tcp_flags_value();
 }
 
-template <>
+template<>
 rc_t
 l3_update_cmd::issue(connection& con)
 {
@@ -61,8 +64,8 @@ l3_update_cmd::issue(connection& con)
   payload.acl_index = m_hw_item.data().value();
   payload.count = m_rules.size();
   memset(payload.tag, 0, sizeof(payload.tag));
-  memcpy(payload.tag, m_key.c_str(),
-         std::min(m_key.length(), sizeof(payload.tag)));
+  memcpy(
+    payload.tag, m_key.c_str(), std::min(m_key.length(), sizeof(payload.tag)));
 
   auto it = m_rules.cbegin();
 
@@ -82,7 +85,7 @@ l3_update_cmd::issue(connection& con)
   return rc_t::OK;
 }
 
-template <>
+template<>
 rc_t
 l3_delete_cmd::issue(connection& con)
 {
@@ -101,7 +104,7 @@ l3_delete_cmd::issue(connection& con)
   return rc_t::OK;
 }
 
-template <>
+template<>
 rc_t
 l3_dump_cmd::issue(connection& con)
 {
@@ -117,7 +120,7 @@ l3_dump_cmd::issue(connection& con)
   return rc_t::OK;
 }
 
-template <>
+template<>
 rc_t
 l2_update_cmd::issue(connection& con)
 {
@@ -128,8 +131,8 @@ l2_update_cmd::issue(connection& con)
   // payload.acl_index = m_hw_item.data().value();
   payload.count = m_rules.size();
   memset(payload.tag, 0, sizeof(payload.tag));
-  memcpy(payload.tag, m_key.c_str(),
-         std::min(m_key.length(), sizeof(payload.tag)));
+  memcpy(
+    payload.tag, m_key.c_str(), std::min(m_key.length(), sizeof(payload.tag)));
 
   auto it = m_rules.cbegin();
 
@@ -149,7 +152,7 @@ l2_update_cmd::issue(connection& con)
   return rc_t::OK;
 }
 
-template <>
+template<>
 rc_t
 l2_delete_cmd::issue(connection& con)
 {
@@ -168,7 +171,7 @@ l2_delete_cmd::issue(connection& con)
   return rc_t::OK;
 }
 
-template <>
+template<>
 rc_t
 l2_dump_cmd::issue(connection& con)
 {