Harald Welte has submitted this change and it was merged.
Change subject: msc_mgcp.c: log endpoint name instead of pointer
......................................................................
msc_mgcp.c: log endpoint name instead of pointer
The logtext currently logs the pointer (address) of the string
variable that holds the endpoint name, rather then the endpoint
name itself.
- Use %s instead of %p in format string
Change-Id: I01b3d07aeedd72be60361249a5bf80fbb68b7bb8
---
M src/libmsc/msc_mgcp.c
1 file changed, 3 insertions(+), 3 deletions(-)
Approvals:
Harald Welte: Looks good to me, approved
Jenkins Builder: Verified
diff --git a/src/libmsc/msc_mgcp.c b/src/libmsc/msc_mgcp.c
index 3cd02c2..b0c68a5 100644
--- a/src/libmsc/msc_mgcp.c
+++ b/src/libmsc/msc_mgcp.c
@@ -274,7 +274,7 @@
OSMO_ASSERT(mgcp);
LOGPFSML(fi, LOGL_DEBUG,
- "CRCX/RAN: creating connection for the RAN side on MGW
endpoint:%p...\n", mgcp_ctx->rtp_endpoint);
+ "CRCX/RAN: creating connection for the RAN side on MGW
endpoint:%s...\n", mgcp_ctx->rtp_endpoint);
/* Generate MGCP message string */
mgcp_msg = (struct mgcp_msg) {
@@ -370,7 +370,7 @@
}
LOGPFSML(fi, LOGL_DEBUG,
- "CRCX/CN creating connection for the CN side on MGW
endpoint:%p...\n", mgcp_ctx->rtp_endpoint);
+ "CRCX/CN creating connection for the CN side on MGW
endpoint:%s...\n", mgcp_ctx->rtp_endpoint);
/* Generate MGCP message string */
mgcp_msg = (struct mgcp_msg) {
@@ -733,7 +733,7 @@
OSMO_ASSERT(mgcp);
LOGPFSML(fi, LOGL_DEBUG,
- "DLCX: removing connection for the RAN and CN side on MGW
endpoint:%p...\n", mgcp_ctx->rtp_endpoint);
+ "DLCX: removing connection for the RAN and CN side on MGW
endpoint:%s...\n", mgcp_ctx->rtp_endpoint);
/* Generate MGCP message string */
mgcp_msg = (struct mgcp_msg) {
--
To view, visit https://gerrit.osmocom.org/7762
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I01b3d07aeedd72be60361249a5bf80fbb68b7bb8
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: dexter <pma...@sysmocom.de>
Gerrit-Reviewer: Harald Welte <lafo...@gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
