Harald Welte has submitted this change and it was merged.
Change subject: error log: sccp_scoc.c: log failure to create/resolve conn_id
......................................................................
error log: sccp_scoc.c: log failure to create/resolve conn_id
Tweak the FIXMEs to clarify that we're lacking a reply to the SCCP user besides
log output.
Change-Id: Ib235ff8e264aaf0c2e9794f464a3ba7b54816f3d
---
M src/sccp_scoc.c
1 file changed, 6 insertions(+), 2 deletions(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/sccp_scoc.c b/src/sccp_scoc.c
index e0742f9..3de986d 100644
--- a/src/sccp_scoc.c
+++ b/src/sccp_scoc.c
@@ -1623,7 +1623,9 @@
/* Allocate new connection structure */
conn = conn_create_id(inst, prim->u.connect.conn_id);
if (!conn) {
- /* FIXME: inform user */
+ /* FIXME: inform SCCP user with proper reply */
+ LOGP(DLSCCP, LOGL_ERROR, "Cannot create conn-id for
primitive %s\n",
+ osmo_scu_prim_name(&prim->oph));
goto out;
}
conn->user = scu;
@@ -1635,7 +1637,9 @@
/* Resolve existing connection structure */
conn = conn_find_by_id(inst, scu_prim_conn_id(prim));
if (!conn) {
- /* FIXME: inform user */
+ /* FIXME: inform SCCP user with proper reply */
+ LOGP(DLSCCP, LOGL_ERROR, "Received unknown conn-id %u
for primitive %s\n",
+ scu_prim_conn_id(prim),
osmo_scu_prim_name(&prim->oph));
goto out;
}
break;
--
To view, visit https://gerrit.osmocom.org/5572
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib235ff8e264aaf0c2e9794f464a3ba7b54816f3d
Gerrit-PatchSet: 3
Gerrit-Project: libosmo-sccp
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <[email protected]>
Gerrit-Reviewer: Harald Welte <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <[email protected]>
