Use the size_t modifier to avoid an unnecessary cast

(cherry picked from commit 8f49a55)
OpenSIPS · Feb 24, 2021 · 947d254 · 947d254
1 parent 263b15b
commit 947d254
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/modules/db_postgres/pg_con.c b/modules/db_postgres/pg_con.c
@@ -60,7 +60,7 @@ struct pg_con* db_postgres_new_connection(struct db_id* id)
 				"\n", (unsigned long)sizeof(struct pg_con));
 		return 0;
 	}
-	LM_DBG("%p=pkg_malloc(%lu)\n", ptr, (unsigned long)sizeof(struct pg_con));
+	LM_DBG("%p=pkg_malloc(%zu)\n", ptr, sizeof(struct pg_con));
 
 	memset(ptr, 0, sizeof(struct pg_con));
 	ptr->ref = 1;

diff --git a/modules/gflags/gflags.c b/modules/gflags/gflags.c
@@ -150,7 +150,7 @@ static int fixup_gflags( void** param, int param_no)
 	}
 	if ( myint >= 8*sizeof(*gflags) ) {
 		LM_ERR("flag <%d> out of "
-			"range [0..%lu]\n", myint, ((unsigned long)8*sizeof(*gflags))-1 );
+			"range [0..%zu]\n", myint, 8*sizeof(*gflags)-1);
 		return E_CFG;
 	}
 	/* convert from flag index to flag bitmap */

diff --git a/modules/mediaproxy/mediaproxy.c b/modules/mediaproxy/mediaproxy.c
@@ -1537,7 +1537,7 @@ use_media_proxy(struct sip_msg *msg, char *dialog_id, ice_candidate_data *ice_da
             if (stream.transport != TSupported)
                 continue; // skip streams with unsupported transports
             if (stream.type.len + stream.ip.len + stream.port.len + stream.direction.len + 4 > str_buf.len) {
-                LM_ERR("media stream description is longer than %lu bytes\n", (unsigned long)sizeof(media_str));
+                LM_ERR("media stream description is longer than %zu bytes\n", sizeof(media_str));
                 return -1;
             }
             len = sprintf(str_buf.s, "%.*s:%.*s:%.*s:%.*s:%s,",
@@ -1586,7 +1586,7 @@ use_media_proxy(struct sip_msg *msg, char *dialog_id, ice_candidate_data *ice_da
                    media_relay.len, media_relay.s, media_str);
 
     if (len >= sizeof(request)) {
-        LM_ERR("mediaproxy request is longer than %lu bytes\n", (unsigned long)sizeof(request));
+        LM_ERR("mediaproxy request is longer than %zu bytes\n", sizeof(request));
         return -1;
     }
 
@@ -1773,7 +1773,7 @@ end_media_session(str callid, str from_tag, str to_tag)
                    to_tag.len, to_tag.s);
 
     if (len >= sizeof(request)) {
-        LM_ERR("mediaproxy request is longer than %lu bytes\n", (unsigned long)sizeof(request));
+        LM_ERR("mediaproxy request is longer than %zu bytes\n", sizeof(request));
         return -1;
     }
 

diff --git a/modules/nat_traversal/nat_traversal.c b/modules/nat_traversal/nat_traversal.c
@@ -1617,7 +1617,7 @@ send_keepalive(NAT_Contact *contact)
                    keepalive_params.extra_headers);
 
     if (len >= sizeof(buffer)) {
-        LM_ERR("keepalive message is longer than %lu bytes\n", (unsigned long)sizeof(buffer));
+        LM_ERR("keepalive message is longer than %zu bytes\n", sizeof(buffer));
         return;
     }
 

diff --git a/modules/sql_cacher/sql_cacher.c b/modules/sql_cacher/sql_cacher.c
@@ -316,7 +316,7 @@ static int parse_cache_entry(unsigned int type, void *val)
 			}
 
 			if (new_entry->nr_columns > sizeof(long long) * 8) {
-				LM_WARN("Too many columns, maximum number is %lu\n", sizeof(long long) * 8);
+				LM_WARN("Too many columns, maximum number is %zu\n", sizeof(long long) * 8);
 				goto parse_err;
 			}
 			/* allocate array of columns and actually parse */