laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified
iu_client: Prevent crash if msgb passed to ranap_iu_tx has no dst

Related: OS#5398
Change-Id: I64beecd2c492d6312f60a1cf016ef022cda5252d
---
M src/iu_client.c
1 file changed, 6 insertions(+), 0 deletions(-)

diff --git a/src/iu_client.c b/src/iu_client.c
index da433da..7d29903 100644
--- a/src/iu_client.c
+++ b/src/iu_client.c
@@ -461,6 +461,12 @@
struct msgb *msg;
struct osmo_scu_prim *prim;

+ if (!uectx) {
+ LOGPIU(LOGL_ERROR, "Discarding to-be-transmitted L3 Message as RANAP DT with unset dst SCCP conn_id!\n",
+ uectx->conn_id);
+ return -ENOTCONN;
+ }
+
LOGPIU(LOGL_INFO, "Transmitting L3 Message as RANAP DT (SCCP conn_id %u)\n",
uectx->conn_id);


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

Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I64beecd2c492d6312f60a1cf016ef022cda5252d
Gerrit-Change-Number: 26824
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-MessageType: merged