lynxis lazus submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved
tcap_as_loadshare: improve log lines

Change-Id: Idffe9a02127c587aca4123f12f74b657d6911a80
---
M src/tcap_as_loadshare.c
1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/tcap_as_loadshare.c b/src/tcap_as_loadshare.c
index 79ab0c3..1dc4f71 100644
--- a/src/tcap_as_loadshare.c
+++ b/src/tcap_as_loadshare.c
@@ -503,11 +503,12 @@
}

rc = parse_tcap(as, ie_data->dat, ie_data->len, &parsed);
- LOGPAS(as, DLTCAP, LOGL_DEBUG, "TCAP decoded rc=%d otid=%u dtid=%u\n", rc, parsed.otid, parsed.dtid);
+ LOGPAS(as, DLTCAP, LOGL_DEBUG, "TCAP decoded rc=%d, msg=0x%02x, otid=%u dtid=%u\n",
+ rc, parsed.present, parsed.otid, parsed.dtid);

if (rc <= 0) {
rate_ctr_inc2(as->ctrg, SS7_AS_CTR_RX_TCAP_FAILED);
- LOGPAS(as, DLTCAP, LOGL_NOTICE, "Failed get TCAP otid/dtid.\n");
+ LOGPAS(as, DLTCAP, LOGL_NOTICE, "Failed parse TCAP message to retrieve otid/dtid.\n");
rc = -EINVAL;
goto out_free_sua;
}

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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: Idffe9a02127c587aca4123f12f74b657d6911a80
Gerrit-Change-Number: 42754
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>