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

msc_mgcp.c: Fix several wrong ptr printf fmt

Fixes several of these type of warnings:
include/osmocom/core/fsm.h:123:38: warning: format ‘%x’ expects argument of 
type ‘unsigned int’, but argument 9 has type ‘char *’ [-Wformat=]
   LOGP((fi)->fsm->log_subsys, level, "%s{%s}: " fmt, \
                                      ^
src/libmsc/msc_mgcp.c:277:71: note: format string is defined here
    "CRCX/RAN: creating connection for the RAN side on MGW endpoint:0x%x...\n", 
mgcp_ctx->rtp_endpoint);
                                                                      ~^

Change-Id: I17b7bed8fc39612286ba66f250b6b26da01d38c0
---
M src/libmsc/msc_mgcp.c
1 file changed, 5 insertions(+), 5 deletions(-)


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

diff --git a/src/libmsc/msc_mgcp.c b/src/libmsc/msc_mgcp.c
index 5c615a9..3cd02c2 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:0x%x...\n", mgcp_ctx->rtp_endpoint);
+                "CRCX/RAN: creating connection for the RAN side on MGW 
endpoint:%p...\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:0x%x...\n", mgcp_ctx->rtp_endpoint);
+                "CRCX/CN creating connection for the CN side on MGW 
endpoint:%p...\n", mgcp_ctx->rtp_endpoint);
 
        /* Generate MGCP message string */
        mgcp_msg = (struct mgcp_msg) {
@@ -528,7 +528,7 @@
        }
 
        LOGPFSML(fi, LOGL_DEBUG,
-                "MDCX/CN: completing connection for the CN side on MGW 
endpoint:0x%x, remote leg expects RTP input on address %s:%u\n",
+                "MDCX/CN: completing connection for the CN side on MGW 
endpoint:%p, remote leg expects RTP input on address %s:%u\n",
                 mgcp_ctx->rtp_endpoint, conn->rtp.remote_addr_cn, 
conn->rtp.remote_port_cn);
 
        /* Generate MGCP message string */
@@ -645,7 +645,7 @@
        }
 
        LOGPFSML(fi, LOGL_DEBUG,
-                "MDCX/RAN: completing connection for the CN side on MGW 
endpoint:0x%x, RAN expects RTP input on address %s:%u\n",
+                "MDCX/RAN: completing connection for the CN side on MGW 
endpoint:%p, RAN expects RTP input on address %s:%u\n",
                 mgcp_ctx->rtp_endpoint, conn->rtp.remote_addr_ran, 
conn->rtp.remote_port_ran);
 
        /* Generate MGCP message string */
@@ -733,7 +733,7 @@
        OSMO_ASSERT(mgcp);
 
        LOGPFSML(fi, LOGL_DEBUG,
-                "DLCX: removing connection for the RAN and CN side on MGW 
endpoint:0x%x...\n", mgcp_ctx->rtp_endpoint);
+                "DLCX: removing connection for the RAN and CN side on MGW 
endpoint:%p...\n", mgcp_ctx->rtp_endpoint);
 
        /* Generate MGCP message string */
        mgcp_msg = (struct mgcp_msg) {

-- 
To view, visit https://gerrit.osmocom.org/7332
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I17b7bed8fc39612286ba66f250b6b26da01d38c0
Gerrit-PatchSet: 1
Gerrit-Project: osmo-msc
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol <pes...@sysmocom.de>

Reply via email to