diff options
| author | Max <msuraev@sysmocom.de> | 2022-07-27 20:41:00 +0700 |
|---|---|---|
| committer | msuraev <msuraev@sysmocom.de> | 2022-08-01 13:53:37 +0000 |
| commit | 1470d0b870c26cb2e0c6e1766f4a5a2133171e56 (patch) | |
| tree | 9dec87f73bb3cbb56ac7e3c4f06b01d11a1a04de | |
| parent | 7038dd0e05177d52f3790467532e70428670a617 (diff) | |
ESME: use osmo_sock_get_name() for logging
This allows us to drop single-use parameters from osmo_esme to facilitate further code changes.
Related: OS#5568
Change-Id: I34bd4c145b0f6287a323e2350808feb59f1d3187
| -rw-r--r-- | src/libmsc/smpp_smsc.c | 3 | ||||
| -rw-r--r-- | src/libmsc/smpp_smsc.h | 2 | ||||
| -rw-r--r-- | src/libmsc/smpp_vty.c | 10 |
3 files changed, 2 insertions, 13 deletions
diff --git a/src/libmsc/smpp_smsc.c b/src/libmsc/smpp_smsc.c index 4ba52fc77..da6607421 100644 --- a/src/libmsc/smpp_smsc.c +++ b/src/libmsc/smpp_smsc.c @@ -947,9 +947,6 @@ static int link_accept_cb(struct smsc *smsc, int fd, esme->wqueue.read_cb = esme_link_read_cb; esme->wqueue.write_cb = esme_link_write_cb; - esme->sa_len = OSMO_MIN(sizeof(esme->sa), s_len); - memcpy(&esme->sa, s, esme->sa_len); - llist_add_tail(&esme->list, &smsc->esme_list); return 0; diff --git a/src/libmsc/smpp_smsc.h b/src/libmsc/smpp_smsc.h index 257589e2d..76772a3ee 100644 --- a/src/libmsc/smpp_smsc.h +++ b/src/libmsc/smpp_smsc.h @@ -45,8 +45,6 @@ struct osmo_esme { uint32_t own_seq_nr; struct osmo_wqueue wqueue; - struct sockaddr_storage sa; - socklen_t sa_len; enum esme_read_state read_state; uint32_t read_len; diff --git a/src/libmsc/smpp_vty.c b/src/libmsc/smpp_vty.c index 31e209085..da260a895 100644 --- a/src/libmsc/smpp_vty.c +++ b/src/libmsc/smpp_vty.c @@ -30,6 +30,7 @@ #include <osmocom/core/linuxlist.h> #include <osmocom/core/utils.h> +#include <osmocom/core/socket.h> #include <osmocom/core/talloc.h> #include <osmocom/msc/vty.h> @@ -524,17 +525,10 @@ DEFUN(cfg_esme_no_alert_notif, cfg_esme_no_alert_notif_cmd, static void dump_one_esme(struct vty *vty, struct osmo_esme *esme) { - char host[128], serv[128]; - - host[0] = 0; - serv[0] = 0; - getnameinfo((const struct sockaddr *) &esme->sa, esme->sa_len, - host, sizeof(host), serv, sizeof(serv), NI_NUMERICSERV); - vty_out(vty, "ESME System ID: %s, Password: %s, SMPP Version %02x%s", esme->system_id, esme->acl ? esme->acl->passwd : "", esme->smpp_version, VTY_NEWLINE); - vty_out(vty, " Connected from: %s:%s%s", host, serv, VTY_NEWLINE); + vty_out(vty, " Connection %s%s", osmo_sock_get_name(tall_vty_ctx, esme->wqueue.bfd.fd), VTY_NEWLINE); if (esme->smsc->def_route == esme->acl) vty_out(vty, " Is current default route%s", VTY_NEWLINE); } |