laforge has submitted this change. (
https://gerrit.osmocom.org/c/osmo-iuh/+/26824 )
Change subject: iu_client: Prevent crash if msgb passed to ranap_iu_tx has no dst
......................................................................
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(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
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
https://gerrit.osmocom.org/c/osmo-iuh/+/26824
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I64beecd2c492d6312f60a1cf016ef022cda5252d
Gerrit-Change-Number: 26824
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: merged