pespin submitted this change.

View Change

Approvals: Jenkins Builder: Verified osmith: Looks good to me, approved
rtp: fail if IuUP instance is not created

Related: Coverity CID#562674
Change-Id: Ic2c8352afe83586d1a5f8cb24f51ae04405a01c2
---
M src/osmo-hnodeb/rtp.c
1 file changed, 4 insertions(+), 0 deletions(-)

diff --git a/src/osmo-hnodeb/rtp.c b/src/osmo-hnodeb/rtp.c
index 6213eee..98da795 100644
--- a/src/osmo-hnodeb/rtp.c
+++ b/src/osmo-hnodeb/rtp.c
@@ -120,6 +120,10 @@

snprintf(iuup_id, sizeof(iuup_id), "ue-%u", conn->ue->conn_id);
conn->iui = osmo_iuup_instance_alloc(conn, iuup_id);
+ if (!conn->iui) {
+ talloc_free(conn);
+ return NULL;
+ }
osmo_iuup_instance_set_user_prim_cb(conn->iui, _iuup_user_prim_cb, conn);
osmo_iuup_instance_set_transport_prim_cb(conn->iui, _iuup_transport_prim_cb, conn);


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

Gerrit-MessageType: merged
Gerrit-Project: osmo-hnodeb
Gerrit-Branch: master
Gerrit-Change-Id: Ic2c8352afe83586d1a5f8cb24f51ae04405a01c2
Gerrit-Change-Number: 43264
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>