osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-bsc-nat/+/27702 )

Change subject: bssap: fix printing bssmap names ......................................................................

bssap: fix printing bssmap names

Use the proper function to print BSSMAP names, so it doesn't say "unknown 0x52" etc.

Related: SYS#5560 Change-Id: I353a9d13b14de6d0ba9aabe8cb3057dd7a98cb51 --- M src/osmo-bsc-nat/bssap.c 1 file changed, 3 insertions(+), 3 deletions(-)

Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve fixeria: Looks good to me, approved laforge: Looks good to me, approved

diff --git a/src/osmo-bsc-nat/bssap.c b/src/osmo-bsc-nat/bssap.c index a7f629e..67a06fb 100644 --- a/src/osmo-bsc-nat/bssap.c +++ b/src/osmo-bsc-nat/bssap.c @@ -63,7 +63,7 @@ ret = bssap_cn_handle_reset_ack(addr, msg, length); break; default: - LOGP(DMAIN, LOGL_ERROR, "%s(%s) is not implemented!\n", __func__, gsm0808_bssap_name(msg->l3h[0])); + LOGP(DMAIN, LOGL_ERROR, "%s(%s) is not implemented!\n", __func__, gsm0808_bssmap_name(msg->l3h[0])); break; }

@@ -96,7 +96,7 @@ ret = bssap_ran_handle_reset(addr, msg, length); break; default: - LOGP(DMAIN, LOGL_ERROR, "%s(%s) is not implemented!\n", __func__, gsm0808_bssap_name(msg->l3h[0])); + LOGP(DMAIN, LOGL_ERROR, "%s(%s) is not implemented!\n", __func__, gsm0808_bssmap_name(msg->l3h[0])); break; }

@@ -111,7 +111,7 @@ return -1; }

- LOGP(DMAIN, LOGL_NOTICE, "Rx UDT BSSMAP %s\n", gsm0808_bssap_name(msg->l3h[0])); + LOGP(DMAIN, LOGL_NOTICE, "Rx UDT BSSMAP %s\n", gsm0808_bssmap_name(msg->l3h[0]));

if (sccp_inst == g_bsc_nat->cn.sccp_inst) return bssap_cn_rcvmsg_udt(addr, msg, length);