osmith submitted this change.

View Change

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
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(-)

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);

To view, visit change 27702. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: osmo-bsc-nat
Gerrit-Branch: master
Gerrit-Change-Id: I353a9d13b14de6d0ba9aabe8cb3057dd7a98cb51
Gerrit-Change-Number: 27702
Gerrit-PatchSet: 1
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged