This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.
laforge gerrit-no-reply at lists.osmocom.orglaforge has submitted this change. ( https://gerrit.osmocom.org/c/libosmo-sccp/+/15830 ) Change subject: Log correct protocol during xua_accept_cb() ...................................................................... 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 --- M src/osmo_ss7.c 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified 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 @@ 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 @@ 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; -- To view, visit https://gerrit.osmocom.org/c/libosmo-sccp/+/15830 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-sccp Gerrit-Branch: master Gerrit-Change-Id: Ic34ddbcd528cd871d9772665e1d0863896f33203 Gerrit-Change-Number: 15830 Gerrit-PatchSet: 2 Gerrit-Owner: laforge <laforge at osmocom.org> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge at osmocom.org> Gerrit-Reviewer: pespin <pespin at sysmocom.de> Gerrit-MessageType: merged -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20191023/44d82a11/attachment.htm>