diff options
| author | Harald Welte <laforge@osmocom.org> | 2019-10-22 21:36:43 +0200 |
|---|---|---|
| committer | laforge <laforge@osmocom.org> | 2019-10-23 18:11:17 +0000 |
| commit | d98960d18972a5fcbcee448ac6742be20e0bec41 (patch) | |
| tree | fd8a5ffabe5897e4eb0aece11345349818682a29 | |
| parent | 1870752ca7c1665731750d2c8288c148cf348b56 (diff) | |
Log correct protocol during xua_accept_cb()
This function is used for both actual SIGTRAN (M3UA, SUA over SCTP)
as well as for IPA/SCCPLITE (over TCP). Having a static "SCTP"
string in the log lines is confusing.
Change-Id: Ic34ddbcd528cd871d9772665e1d0863896f33203
| -rw-r--r-- | src/osmo_ss7.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/osmo_ss7.c b/src/osmo_ss7.c index 88c4922..464f2f4 100644 --- a/src/osmo_ss7.c +++ b/src/osmo_ss7.c @@ -1710,9 +1710,9 @@ static int xua_accept_cb(struct osmo_stream_srv_link *link, int fd) struct osmo_stream_srv *srv; struct osmo_ss7_asp *asp; char *sock_name = osmo_sock_get_name(link, fd); + const char *proto_name = get_value_string(osmo_ss7_asp_protocol_vals, oxs->cfg.proto); - LOGP(DLSS7, LOGL_INFO, "%s: New %s connection accepted\n", - sock_name, get_value_string(osmo_ss7_asp_protocol_vals, oxs->cfg.proto)); + LOGP(DLSS7, LOGL_INFO, "%s: New %s connection accepted\n", sock_name, proto_name); if (oxs->cfg.proto == OSMO_SS7_ASP_PROT_IPA) { srv = osmo_stream_srv_create(oxs, link, fd, @@ -1737,9 +1737,9 @@ static int xua_accept_cb(struct osmo_stream_srv_link *link, int fd) sock_name, asp->cfg.name); } else { if (!oxs->cfg.accept_dyn_reg) { - LOGP(DLSS7, LOGL_NOTICE, "%s: SCTP connection without matching " + LOGP(DLSS7, LOGL_NOTICE, "%s: %s connection without matching " "ASP definition and no dynamic registration enabled, terminating\n", - sock_name); + sock_name, proto_name); } else { char namebuf[32]; static uint32_t dyn_asp_num = 0; |