<p>fixeria has uploaded this change for <strong>review</strong>.</p><p><a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/20444">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">vty: use install_lib_element() and install_lib_element_ve()<br><br>See https://lists.osmocom.org/pipermail/openbsc/2020-October/013278.html.<br><br>Change-Id: I727e27f4d4d9550e34cb0073134a9ed7faae3c66<br>Related: SYS#4937<br>---<br>M src/osmo_ss7_vty.c<br>M src/sccp_vty.c<br>2 files changed, 70 insertions(+), 70 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://gerrit.osmocom.org:29418/libosmo-sccp refs/changes/44/20444/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/osmo_ss7_vty.c b/src/osmo_ss7_vty.c</span><br><span>index 08e650e..2692209 100644</span><br><span>--- a/src/osmo_ss7_vty.c</span><br><span>+++ b/src/osmo_ss7_vty.c</span><br><span>@@ -1933,85 +1933,85 @@</span><br><span> static void vty_init_addr(void)</span><br><span> {</span><br><span>       install_node(&sccpaddr_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element_ve(&cs7_show_sccpaddr_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_NODE, &cs7_sccpaddr_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element(L_CS7_NODE, &cs7_sccpaddr_del_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_pc_del_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ssn_del_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element_ve(&cs7_show_sccpaddr_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_NODE, &cs7_sccpaddr_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_NODE, &cs7_sccpaddr_del_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_pc_del_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ssn_del_cmd);</span><br><span> #if 0</span><br><span>    /* FIXME: IP-Address based SCCP-Routing is currently not supported,</span><br><span>   * so we leave the related VTY options out for now */</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ip_del_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ip_del_cmd);</span><br><span> #endif</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_gt_del_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ri_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_pc_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ssn_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_gt_del_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ri_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_pc_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ssn_cmd);</span><br><span> #if 0</span><br><span>        /* FIXME: IP-Address based SCCP-Routing is currently not supported,</span><br><span>   * so we leave the related VTY options out for now */</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ipv4_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ipv6_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ipv4_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_ipv6_cmd);</span><br><span> #endif</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_gt_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_SCCPADDR_NODE, &cs7_sccpaddr_gt_cmd);</span><br><span>  install_node(&sccpaddr_gt_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_gti_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_tt_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_npi_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_nai_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_digits_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_gti_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_tt_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_npi_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_nai_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_SCCPADDR_GT_NODE, &cs7_sccpaddr_gt_digits_cmd);</span><br><span> }</span><br><span> </span><br><span> static void vty_init_shared(void *ctx)</span><br><span> {</span><br><span>        g_ctx = ctx;</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        install_element_ve(&show_cs7_user_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element_ve(&show_cs7_xua_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element_ve(&show_cs7_config_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element_ve(&show_cs7_user_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element_ve(&show_cs7_xua_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+        install_lib_element_ve(&show_cs7_config_cmd);</span><br><span> </span><br><span>        /* the mother of all VTY config nodes */</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(CONFIG_NODE, &cs7_instance_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(CONFIG_NODE, &cs7_instance_cmd);</span><br><span> </span><br><span>         install_node(&cs7_node, config_write_cs7);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_NODE, &cs7_net_ind_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element(L_CS7_NODE, &cs7_point_code_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_NODE, &cs7_pc_format_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-    install_element(L_CS7_NODE, &cs7_pc_format_def_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_NODE, &cs7_pc_delimiter_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_NODE, &cs7_permit_dyn_rkm_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_NODE, &cs7_net_ind_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+        install_lib_element(L_CS7_NODE, &cs7_point_code_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_NODE, &cs7_pc_format_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element(L_CS7_NODE, &cs7_pc_format_def_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_NODE, &cs7_pc_delimiter_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_NODE, &cs7_permit_dyn_rkm_cmd);</span><br><span> </span><br><span>    install_node(&asp_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element_ve(&show_cs7_asp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element(L_CS7_NODE, &cs7_asp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_NODE, &no_cs7_asp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_ASP_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element(L_CS7_ASP_NODE, &asp_remote_ip_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_ASP_NODE, &asp_local_ip_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_ASP_NODE, &asp_qos_class_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_ASP_NODE, &asp_role_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element(L_CS7_ASP_NODE, &asp_sctp_role_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_ASP_NODE, &asp_block_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-    install_element(L_CS7_ASP_NODE, &asp_shutdown_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element_ve(&show_cs7_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+        install_lib_element(L_CS7_NODE, &cs7_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_NODE, &no_cs7_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_ASP_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+        install_lib_element(L_CS7_ASP_NODE, &asp_remote_ip_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_ASP_NODE, &asp_local_ip_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_ASP_NODE, &asp_qos_class_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_ASP_NODE, &asp_role_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_ASP_NODE, &asp_sctp_role_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_ASP_NODE, &asp_block_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element(L_CS7_ASP_NODE, &asp_shutdown_cmd);</span><br><span> </span><br><span>      install_node(&as_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element_ve(&show_cs7_as_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_NODE, &cs7_as_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_NODE, &no_cs7_as_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_AS_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_AS_NODE, &as_asp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_AS_NODE, &as_no_asp_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-     install_element(L_CS7_AS_NODE, &as_traf_mode_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_AS_NODE, &as_no_traf_mode_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_AS_NODE, &as_recov_tout_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_AS_NODE, &as_qos_class_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_AS_NODE, &as_rout_key_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_AS_NODE, &as_rout_key_si_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_AS_NODE, &as_rout_key_ssn_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_AS_NODE, &as_rout_key_si_ssn_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-    install_element(L_CS7_AS_NODE, &as_pc_override_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_AS_NODE, &as_pc_patch_sccp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element_ve(&show_cs7_as_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_NODE, &cs7_as_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_NODE, &no_cs7_as_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_AS_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_AS_NODE, &as_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_AS_NODE, &as_no_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+       install_lib_element(L_CS7_AS_NODE, &as_traf_mode_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_AS_NODE, &as_no_traf_mode_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_AS_NODE, &as_recov_tout_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_AS_NODE, &as_qos_class_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_AS_NODE, &as_rout_key_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_AS_NODE, &as_rout_key_si_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_AS_NODE, &as_rout_key_ssn_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_AS_NODE, &as_rout_key_si_ssn_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element(L_CS7_AS_NODE, &as_pc_override_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_AS_NODE, &as_pc_patch_sccp_cmd);</span><br><span> </span><br><span>   vty_init_addr();</span><br><span> }</span><br><span>@@ -2028,15 +2028,15 @@</span><br><span>      vty_init_shared(ctx);</span><br><span> </span><br><span>    install_node(&rtable_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element_ve(&show_cs7_route_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-    install_element(L_CS7_NODE, &cs7_route_table_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_RTABLE_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_RTABLE_NODE, &cs7_rt_upd_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element(L_CS7_RTABLE_NODE, &cs7_rt_rem_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element_ve(&show_cs7_route_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+      install_lib_element(L_CS7_NODE, &cs7_route_table_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_RTABLE_NODE, &cfg_description_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_RTABLE_NODE, &cs7_rt_upd_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element(L_CS7_RTABLE_NODE, &cs7_rt_rem_cmd);</span><br><span> </span><br><span>     install_node(&xua_node, NULL);</span><br><span style="color: hsl(0, 100%, 40%);">-      install_element(L_CS7_NODE, &cs7_xua_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-  install_element(L_CS7_NODE, &no_cs7_xua_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-       install_element(L_CS7_XUA_NODE, &xua_local_ip_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">- install_element(L_CS7_XUA_NODE, &xua_accept_dyn_asp_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_NODE, &cs7_xua_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+    install_lib_element(L_CS7_NODE, &no_cs7_xua_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+ install_lib_element(L_CS7_XUA_NODE, &xua_local_ip_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element(L_CS7_XUA_NODE, &xua_accept_dyn_asp_cmd);</span><br><span> }</span><br><span>diff --git a/src/sccp_vty.c b/src/sccp_vty.c</span><br><span>index 46afb49..2e5b345 100644</span><br><span>--- a/src/sccp_vty.c</span><br><span>+++ b/src/sccp_vty.c</span><br><span>@@ -255,11 +255,11 @@</span><br><span> </span><br><span> void osmo_sccp_vty_init(void)</span><br><span> {</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element_ve(&show_sccp_users_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element_ve(&show_sccp_user_ssn_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-        install_element_ve(&show_sccp_connections_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+   install_lib_element_ve(&show_sccp_users_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element_ve(&show_sccp_user_ssn_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+  install_lib_element_ve(&show_sccp_connections_cmd);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-     install_element_ve(&show_sccp_timers_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+        install_lib_element_ve(&show_sccp_timers_cmd);</span><br><span>   gen_sccp_timer_cmd_strs(&sccp_timer_cmd);</span><br><span style="color: hsl(0, 100%, 40%);">-   install_element(L_CS7_NODE, &sccp_timer_cmd);</span><br><span style="color: hsl(120, 100%, 40%);">+     install_lib_element(L_CS7_NODE, &sccp_timer_cmd);</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://gerrit.osmocom.org/c/libosmo-sccp/+/20444">change 20444</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.osmocom.org/c/libosmo-sccp/+/20444"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-sccp </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: I727e27f4d4d9550e34cb0073134a9ed7faae3c66 </div>
<div style="display:none"> Gerrit-Change-Number: 20444 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de> </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>