From 607275c30be14266c4f1724b29a7cddea2062802 Mon Sep 17 00:00:00 2001
From: Neels Hofmeyr <neels@hofmeyr.de>
Date: Mon, 25 Sep 2017 00:35:06 +0200
Subject: gb: drop special vty exit commands, use vty_install_default()

L_NS_NODE and L_BSSGP_NODE had specialized 'exit' and 'end' vty commands, but
all they do is return to the CONFIG and ENABLE_NODEs like the default 'exit'
and 'end' commands. Drop them and use the default 'exit' and 'end' cmds.

Examining BSSGP and NS node behavior in osmo-sgsn exhibited identical list and
exit/end behavior before and after this patch.

Prepares for an upcoming commit incorporating vty_install_default() into
install_node(), see I5021c64a787b63314e0f2f1cba0b8fc7bff4f09b: this patch
changes to the default commands, the upcoming change implies them.

Change-Id: I5b0de066b4249d482c22620d5b1bcb03f381293c
---
 src/gb/common_vty.c     | 34 ----------------------------------
 src/gb/common_vty.h     |  3 ---
 src/gb/gprs_bssgp_vty.c |  4 +---
 src/gb/gprs_ns_vty.c    |  4 +---
 4 files changed, 2 insertions(+), 43 deletions(-)

(limited to 'src/gb')

diff --git a/src/gb/common_vty.c b/src/gb/common_vty.c
index 5de51355..16f3bbd8 100644
--- a/src/gb/common_vty.c
+++ b/src/gb/common_vty.c
@@ -35,40 +35,6 @@
 
 #include "common_vty.h"
 
-/* Down vty node level. */
-gDEFUN(libgb_exit,
-       libgb_exit_cmd, "exit", "Exit current mode and down to previous mode\n")
-{
-	switch (vty->node) {
-	case L_NS_NODE:
-	case L_BSSGP_NODE:
-		vty->node = CONFIG_NODE;
-		vty->index = NULL;
-		break;
-	default:
-		break;
-	}
-	return CMD_SUCCESS;
-}
-
-/* End of configuration. */
-gDEFUN(libgb_end,
-       libgb_end_cmd, "end", "End current mode and change to enable mode.")
-{
-	switch (vty->node) {
-	case L_NS_NODE:
-	case L_BSSGP_NODE:
-		vty_config_unlock(vty);
-		vty->node = ENABLE_NODE;
-		vty->index = NULL;
-		vty->index_sub = NULL;
-		break;
-	default:
-		break;
-	}
-	return CMD_SUCCESS;
-}
-
 int gprs_log_filter_fn(const struct log_context *ctx,
 			struct log_target *tar)
 {
diff --git a/src/gb/common_vty.h b/src/gb/common_vty.h
index a0674a7f..801d2dab 100644
--- a/src/gb/common_vty.h
+++ b/src/gb/common_vty.h
@@ -5,6 +5,3 @@
 
 extern int DNS, DBSSGP;
 
-extern struct cmd_element libgb_exit_cmd;
-extern struct cmd_element libgb_end_cmd;
-
diff --git a/src/gb/gprs_bssgp_vty.c b/src/gb/gprs_bssgp_vty.c
index 30ba6037..dc786a14 100644
--- a/src/gb/gprs_bssgp_vty.c
+++ b/src/gb/gprs_bssgp_vty.c
@@ -211,9 +211,7 @@ int bssgp_vty_init(void)
 
 	install_element(CONFIG_NODE, &cfg_bssgp_cmd);
 	install_node(&bssgp_node, config_write_bssgp);
-	install_default(L_BSSGP_NODE);
-	install_element(L_BSSGP_NODE, &libgb_exit_cmd);
-	install_element(L_BSSGP_NODE, &libgb_end_cmd);
+	vty_install_default(L_BSSGP_NODE);
 
 	return 0;
 }
diff --git a/src/gb/gprs_ns_vty.c b/src/gb/gprs_ns_vty.c
index e320ba29..4bd50970 100644
--- a/src/gb/gprs_ns_vty.c
+++ b/src/gb/gprs_ns_vty.c
@@ -594,9 +594,7 @@ int gprs_ns_vty_init(struct gprs_ns_inst *nsi)
 
 	install_element(CONFIG_NODE, &cfg_ns_cmd);
 	install_node(&ns_node, config_write_ns);
-	install_default(L_NS_NODE);
-	install_element(L_NS_NODE, &libgb_exit_cmd);
-	install_element(L_NS_NODE, &libgb_end_cmd);
+	vty_install_default(L_NS_NODE);
 	install_element(L_NS_NODE, &cfg_nse_nsvci_cmd);
 	install_element(L_NS_NODE, &cfg_nse_remoteip_cmd);
 	install_element(L_NS_NODE, &cfg_nse_remoteport_cmd);
-- 
cgit v1.2.3