Change in libosmocore[master]: vty: Introduce telnet_init_default and make vty port configurable

[Holger Freyther]

Holger Freyther has uploaded this change for review. ( 
https://gerrit.osmocom.org/12321


Change subject: vty: Introduce telnet_init_default and make vty port 
configurable
......................................................................

vty: Introduce telnet_init_default and make vty port configurable

Make configuring the bind port optional in the line vty/bind command
and introduce telnet_init_default to automatically consume this value.

Client code should use it like:

        int rc = telnet_init_default(ctx, priv, OSMO_VTY_PORT_SGSN);

Change-Id: Id5fb2faaf4311bd7284ee870526a6f87b7e260f3
---
M include/osmocom/vty/telnet_interface.h
M include/osmocom/vty/vty.h
M src/vty/telnet_interface.c
M src/vty/vty.c
4 files changed, 34 insertions(+), 3 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/21/12321/1

diff --git a/include/osmocom/vty/telnet_interface.h 
b/include/osmocom/vty/telnet_interface.h
index d653466..da7cf83 100644
--- a/include/osmocom/vty/telnet_interface.h
+++ b/include/osmocom/vty/telnet_interface.h
@@ -47,6 +47,7 @@

 int telnet_init(void *tall_ctx, void *priv, int port);
 int telnet_init_dynif(void *tall_ctx, void *priv, const char *ip, int port);
+int telnet_init_default(void *tall_ctx, void *priv, int default_port);

 void telnet_exit(void);

diff --git a/include/osmocom/vty/vty.h b/include/osmocom/vty/vty.h
index 02ba03e..c4cf707 100644
--- a/include/osmocom/vty/vty.h
+++ b/include/osmocom/vty/vty.h
@@ -210,6 +210,8 @@

 /* Return IP address passed to the 'line vty'/'bind' command, or "127.0.0.1" */
 const char *vty_get_bind_addr(void);
+/** Returns configured port passed to the 'line vty'/'bind' command or 
default_port. */
+int vty_get_bind_port(int default_port);

 extern void *tall_vty_ctx;

diff --git a/src/vty/telnet_interface.c b/src/vty/telnet_interface.c
index fcb4c8d..031bfab 100644
--- a/src/vty/telnet_interface.c
+++ b/src/vty/telnet_interface.c
@@ -105,6 +105,18 @@
        return 0;
 }

+/*! Initializes telnet based VTY interface using the configured bind addr/port.
+ *  \param[in] tall_ctx \ref talloc context
+ *  \param[in] priv private data to be passed to callback
+ *  \param[in] default_port UDP port number to use if not explicitely 
configured
+ */
+int telnet_init_default(void *tall_ctx, void *priv, int default_port)
+{
+       return telnet_init_dynif(tall_ctx, priv, vty_get_bind_addr(),
+                                vty_get_bind_port(default_port));
+}
+
+
 extern struct host host;

 /*! close a telnet connection */
diff --git a/src/vty/vty.c b/src/vty/vty.c
index 70f6811..7f6c225 100644
--- a/src/vty/vty.c
+++ b/src/vty/vty.c
@@ -90,6 +90,8 @@
  * use NULL and VTY_BIND_ADDR_DEFAULT instead. */
 static const char *vty_bind_addr = NULL;
 #define VTY_BIND_ADDR_DEFAULT "127.0.0.1"
+/* Port the VTY should bind to. -1 means not configured */
+static int vty_bind_port = -1;

 /* Configure lock. */
 static int vty_config;
@@ -1612,12 +1614,13 @@
 }

 /* vty bind */
-DEFUN(vty_bind, vty_bind_cmd, "bind A.B.C.D",
+DEFUN(vty_bind, vty_bind_cmd, "bind A.B.C.D [<0-65535>]",
       "Accept VTY telnet connections on local interface\n"
       "Local interface IP address (default: " VTY_BIND_ADDR_DEFAULT ")\n")
 {
        talloc_free((void*)vty_bind_addr);
        vty_bind_addr = talloc_strdup(tall_vty_ctx, argv[0]);
+       vty_bind_port = argc > 1 ? atoi(argv[1]) : -1;
        return CMD_SUCCESS;
 }

@@ -1628,6 +1631,13 @@
        return vty_bind_addr;
 }

+int vty_get_bind_port(int default_port)
+{
+       if (vty_bind_port >= 0)
+               return vty_bind_port;
+       return default_port;
+}
+
 DEFUN(service_advanced_vty,
       service_advanced_vty_cmd,
       "service advanced-vty",
@@ -1700,8 +1710,14 @@
                vty_out(vty, " login%s", VTY_NEWLINE);

        /* bind */
-       if (vty_bind_addr && (strcmp(vty_bind_addr, VTY_BIND_ADDR_DEFAULT) != 
0))
-               vty_out(vty, " bind %s%s", vty_bind_addr, VTY_NEWLINE);
+       if (vty_bind_addr && (strcmp(vty_bind_addr, VTY_BIND_ADDR_DEFAULT) != 0 
|| vty_bind_port >= 0)) {
+               if (vty_bind_port >= 0) {
+                       vty_out(vty, " bind %s %d%s", vty_bind_addr,
+                               vty_bind_port, VTY_NEWLINE);
+               } else {
+                       vty_out(vty, " bind %s%s", vty_bind_addr, VTY_NEWLINE);
+               }
+       }

        vty_out(vty, "!%s", VTY_NEWLINE);


--
To view, visit https://gerrit.osmocom.org/12321
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5fb2faaf4311bd7284ee870526a6f87b7e260f3
Gerrit-Change-Number: 12321
Gerrit-PatchSet: 1
Gerrit-Owner: Holger Freyther <hol...@freyther.de>