[sr-dev] git:master:7d9d9fe2: ndb_redis: fix compilation warnings

Victor Seva linuxmaniac at torreviejawireless.org
Thu Dec 20 00:17:40 CET 2018


Module: kamailio
Branch: master
Commit: 7d9d9fe21a121a38d4a3861ce252a1a485a35c71
URL: https://github.com/kamailio/kamailio/commit/7d9d9fe21a121a38d4a3861ce252a1a485a35c71

Author: Victor Seva <linuxmaniac at torreviejawireless.org>
Committer: Victor Seva <linuxmaniac at torreviejawireless.org>
Date: 2018-12-20T00:06:45+01:00

ndb_redis: fix compilation warnings

> warning: field precision should have type 'int', but argument has type 'size_t' (aka 'unsigned long') [-Wformat]

---

Modified: src/modules/ndb_redis/redis_client.c

---

Diff:  https://github.com/kamailio/kamailio/commit/7d9d9fe21a121a38d4a3861ce252a1a485a35c71.diff
Patch: https://github.com/kamailio/kamailio/commit/7d9d9fe21a121a38d4a3861ce252a1a485a35c71.patch

---

diff --git a/src/modules/ndb_redis/redis_client.c b/src/modules/ndb_redis/redis_client.c
index f923878284..7e3ddb0d8c 100644
--- a/src/modules/ndb_redis/redis_client.c
+++ b/src/modules/ndb_redis/redis_client.c
@@ -758,7 +758,7 @@ int check_cluster_reply(redisReply *reply, redisc_server_t **rsrv)
 	char spec_new[100];
 
 	if(redis_cluster_param) {
-		LM_DBG("Redis replied: \"%.*s\"\n", reply->len, reply->str);
+		LM_DBG("Redis replied: \"%.*s\"\n", (int)reply->len, reply->str);
 		if((reply->len > 7) && (strncmp(reply->str, "MOVED", 5) == 0)) {
 			port = 6379;
 			if(strchr(reply->str, ':') > 0) {
@@ -769,7 +769,7 @@ int check_cluster_reply(redisReply *reply, redisc_server_t **rsrv)
 				LM_DBG("Port \"%.*s\" [%i] => %i\n", tmpstr.len, tmpstr.s,
 						tmpstr.len, port);
 			} else {
-				LM_ERR("No Port in REDIS MOVED Reply (%.*s)\n", reply->len,
+				LM_ERR("No Port in REDIS MOVED Reply (%.*s)\n", (int)reply->len,
 						reply->str);
 				return 0;
 			}
@@ -778,7 +778,7 @@ int check_cluster_reply(redisReply *reply, redisc_server_t **rsrv)
 				addr.s = strchr(reply->str + 6, ' ') + 1;
 				LM_DBG("Host \"%.*s\" [%i]\n", addr.len, addr.s, addr.len);
 			} else {
-				LM_ERR("No Host in REDIS MOVED Reply (%.*s)\n", reply->len,
+				LM_ERR("No Host in REDIS MOVED Reply (%.*s)\n", (int)reply->len,
 						reply->str);
 				return 0;
 			}




More information about the sr-dev mailing list