fixeria has uploaded this change for review. ( 
https://gerrit.osmocom.org/c/libosmocore/+/14972


Change subject: vty/telnet_interface.c: log connection accept() / close() events
......................................................................

vty/telnet_interface.c: log connection accept() / close() events

Change-Id: I77b31abfa159d2f269deaa5a08d94b7bbba7d23c
---
M src/vty/telnet_interface.c
1 file changed, 13 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.osmocom.org:29418/libosmocore refs/changes/72/14972/1

diff --git a/src/vty/telnet_interface.c b/src/vty/telnet_interface.c
index d44754c..dbbbfec 100644
--- a/src/vty/telnet_interface.c
+++ b/src/vty/telnet_interface.c
@@ -123,6 +123,16 @@
 int telnet_close_client(struct osmo_fd *fd)
 {
        struct telnet_connection *conn = (struct telnet_connection*)fd->data;
+       struct sockaddr_in sockaddr;
+       socklen_t len = sizeof(sockaddr);
+       int rc;
+
+       /* FIXME: getsockname() always fails: "Bad file descriptor" */
+       rc = getsockname(fd->fd, (struct sockaddr *) &sockaddr, &len);
+       if (rc == 0) {
+               LOGP(DLGLOBAL, LOGL_INFO, "Closing telnet connection %s:%u\n",
+                    inet_ntoa(sockaddr.sin_addr), ntohs(sockaddr.sin_port));
+       }

        close(fd->fd);
        osmo_fd_unregister(fd);
@@ -173,6 +183,9 @@
                return new_connection;
        }

+       LOGP(DLGLOBAL, LOGL_INFO, "Accept()ed new telnet connection from 
%s:%u\n",
+            inet_ntoa(sockaddr.sin_addr), ntohs(sockaddr.sin_port));
+
        connection = talloc_zero(tall_telnet_ctx, struct telnet_connection);
        connection->priv = fd->data;
        connection->fd.data = connection;

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

Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I77b31abfa159d2f269deaa5a08d94b7bbba7d23c
Gerrit-Change-Number: 14972
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-MessageType: newchange

Reply via email to