Review at  https://gerrit.osmocom.org/7762

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(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-msc refs/changes/62/7762/1

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: newchange
Gerrit-Change-Id: I01b3d07aeedd72be60361249a5bf80fbb68b7bb8
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: dexter <pma...@sysmocom.de>

Reply via email to