[MINOR] fix several printf formats and missing arguments

Last patch revealed a number of mistakes in printf-like calls, mostly int/long
mismatches, and a few missing arguments.

(cherry-picked from commit 1772ece0252fa517ace6c554ee77ca09c3dda7e8)
(cherry-picked from commit 8169959df85d50356a0beec0c74e23dcae1d7c32)

diff --git a/src/cfgparse.c b/src/cfgparse.c
index e9a3320e289e3fc7d489b9c4510117412ebd01d5..f067832b7d1e641ecd43e1004390fbd540c9f239 100644 (file)
--- a/src/cfgparse.c
+++ b/src/cfgparse.c
@@ -361,7 +361,7 @@ int cfg_parse_global(const char *file, int linenum, char **args, int inv)
        else if (!strcmp(args[0], "group")) {
                struct group *ha_group;
                if (global.gid != 0) {
-                       Alert("parsing [%s:%d] : gid/group was already specified. Continuing.\n", file, linenum, args[0]);
+                       Alert("parsing [%s:%d] : gid/group was already specified. Continuing.\n", file, linenum);
                        return 0;
                }
                errno = 0;
@@ -2361,7 +2361,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        return 0;
 
                if (*(args[2]) == 0) {
-                       Alert("parsing [%s:%d] : <%s> expects <status_code> and <url> as arguments.\n", file, linenum);
+                       Alert("parsing [%s:%d] : <%s> expects <status_code> and <url> as arguments.\n", file, linenum, args[0]);
                        return -1;
                }
 
@@ -2399,7 +2399,7 @@ int cfg_parse_listen(const char *file, int linenum, char **args, int inv)
                        return 0;
 
                if (*(args[2]) == 0) {
-                       Alert("parsing [%s:%d] : <%s> expects <status_code> and <file> as arguments.\n", file, linenum);
+                       Alert("parsing [%s:%d] : <%s> expects <status_code> and <file> as arguments.\n", file, linenum, args[0]);
                        return -1;
                }
 
@@ -2901,8 +2901,8 @@ int readcfgfile(const char *file)
                                /* minconn was not specified, so we set it to maxconn */
                                newsrv->minconn = newsrv->maxconn;
                        } else if (newsrv->minconn != newsrv->maxconn && !curproxy->fullconn) {
-                               Alert("parsing %s, %s '%s' : fullconn is mandatory when minconn is set on a server.\n",
-                                     file, proxy_type_str(curproxy), curproxy->id, linenum);
+                               Alert("parsing [%s:%d] : %s '%s' : fullconn is mandatory when minconn is set on a server.\n",
+                                     file, linenum, proxy_type_str(curproxy), curproxy->id);
                                goto err;
                        }
                        newsrv = newsrv->next;

diff --git a/src/dumpstats.c b/src/dumpstats.c
index 91d46544d81b801e7ec90b2b93b6c78050254706..fc45d4926d580f7cb809bd5b3a279f5a5f891658 100644 (file)
--- a/src/dumpstats.c
+++ b/src/dumpstats.c
@@ -882,7 +882,7 @@ int stats_dump_proxy(struct session *s, struct proxy *px, struct uri_auth *uri)
                                        chunk_printf(&msg, sizeof(trash),
                                             "%d,%d,%d,%d,",
                                             sv->failed_checks, sv->down_trans,
-                                            now.tv_sec - sv->last_change, srv_downtime(sv));
+                                            (int)(now.tv_sec - sv->last_change), srv_downtime(sv));
                                else
                                        chunk_printf(&msg, sizeof(trash),
                                             ",,,,");
@@ -1001,7 +1001,7 @@ int stats_dump_proxy(struct session *s, struct proxy *px, struct uri_auth *uri)
                                     (px->lbprm.tot_weight > 0 || !px->srv) ? "UP" : "DOWN",
                                     (px->lbprm.tot_weight * px->lbprm.wmult + px->lbprm.wdiv - 1) / px->lbprm.wdiv,
                                     px->srv_act, px->srv_bck,
-                                    px->down_trans, now.tv_sec - px->last_change,
+                                    px->down_trans, (int)(now.tv_sec - px->last_change),
                                     px->srv?be_downtime(px):0,
                                     relative_pid, px->uuid,
                                     px->cum_lbconn);

diff --git a/src/haproxy.c b/src/haproxy.c
index 8cecf072632414c2cafda239e99db1b7f4cd3e6c..e0c33a2380184e6a1ce911d25956d49dcb854133 100644 (file)
--- a/src/haproxy.c
+++ b/src/haproxy.c
@@ -1027,7 +1027,7 @@ int main(int argc, char **argv)
        getrlimit(RLIMIT_NOFILE, &limit);
        if (limit.rlim_cur < global.maxsock) {
                Warning("[%s.main()] FD limit (%d) too low for maxconn=%d/maxsock=%d. Please raise 'ulimit-n' to %d or more to avoid any trouble.\n",
-                       argv[0], limit.rlim_cur, global.maxconn, global.maxsock, global.maxsock);
+                       argv[0], (int)limit.rlim_cur, global.maxconn, global.maxsock, global.maxsock);
        }
 
        if (global.mode & MODE_DAEMON) {

diff --git a/src/log.c b/src/log.c
index e306a549ba89a6e43253c77c90a85d65c9150ee6..667b8111d73c392f83f13a17b32211877539e5fe 100644 (file)
--- a/src/log.c
+++ b/src/log.c
@@ -355,14 +355,14 @@ void tcp_sess_log(struct session *s)
        svid = (tolog & LW_SVID) ? (s->srv != NULL) ? s->srv->id : "<NOSRV>" : "-";
 
        send_log(prx_log, LOG_INFO, "%s:%d [%02d/%s/%04d:%02d:%02d:%02d.%03d]"
-                " %s %s/%s %d/%d/%s%d %s%lld"
-                " %c%c %d/%d/%d/%d %d/%d\n",
+                " %s %s/%s %ld/%ld/%s%ld %s%lld"
+                " %c%c %d/%d/%d/%d %ld/%ld\n",
                 pn,
                 (s->cli_addr.ss_family == AF_INET) ?
                 ntohs(((struct sockaddr_in *)&s->cli_addr)->sin_port) :
                 ntohs(((struct sockaddr_in6 *)&s->cli_addr)->sin6_port),
                 tm.tm_mday, monthname[tm.tm_mon], tm.tm_year+1900,
-                tm.tm_hour, tm.tm_min, tm.tm_sec, s->logs.tv_accept.tv_usec/1000,
+                tm.tm_hour, tm.tm_min, tm.tm_sec, (int)s->logs.tv_accept.tv_usec/1000,
                 fe->id, be->id, svid,
                 (s->logs.t_queue >= 0) ? s->logs.t_queue : -1,
                 (s->logs.t_connect >= 0) ? s->logs.t_connect - s->logs.t_queue : -1,

diff --git a/src/memory.c b/src/memory.c
index b157cf0a06b5f732e0465cbb618bca447343267c..7948bf2ff2473cf73c7630f273cc9efef9f37ced 100644 (file)
--- a/src/memory.c
+++ b/src/memory.c
@@ -174,7 +174,7 @@ void dump_pools(void)
        allocated = used = nbpools = 0;
        qfprintf(stderr, "Dumping pools usage.\n");
        list_for_each_entry(entry, &pools, list) {
-               qfprintf(stderr, "  - Pool %s (%d bytes) : %d allocated (%lu bytes), %d used, %d users%s\n",
+               qfprintf(stderr, "  - Pool %s (%d bytes) : %d allocated (%u bytes), %d used, %d users%s\n",
                         entry->name, entry->size, entry->allocated,
                         entry->size * entry->allocated, entry->used,
                         entry->users, (entry->flags & MEM_F_SHARED) ? " [SHARED]" : "");

diff --git a/src/proto_http.c b/src/proto_http.c
index bffe7b91a632260b161fa10316d7598bde0e7600..9fb2344cced0b66152cf6818536c2ce5907d3e3c 100644 (file)
--- a/src/proto_http.c
+++ b/src/proto_http.c
@@ -774,14 +774,14 @@ static void http_sess_log(struct session *s)
 
        send_log(prx_log, LOG_INFO,
                 "%s:%d [%02d/%s/%04d:%02d:%02d:%02d.%03d]"
-                " %s %s/%s %d/%d/%d/%d/%s%d %d %s%lld"
-                " %s %s %c%c%c%c %d/%d/%d/%d %d/%d%s\n",
+                " %s %s/%s %d/%ld/%ld/%ld/%s%ld %d %s%lld"
+                " %s %s %c%c%c%c %d/%d/%d/%d %ld/%ld%s\n",
                 pn,
                 (s->cli_addr.ss_family == AF_INET) ?
                 ntohs(((struct sockaddr_in *)&s->cli_addr)->sin_port) :
                 ntohs(((struct sockaddr_in6 *)&s->cli_addr)->sin6_port),
                 tm.tm_mday, monthname[tm.tm_mon], tm.tm_year+1900,
-                tm.tm_hour, tm.tm_min, tm.tm_sec, s->logs.tv_accept.tv_usec/1000,
+                tm.tm_hour, tm.tm_min, tm.tm_sec, (int)s->logs.tv_accept.tv_usec/1000,
                 fe->id, be->id, svid,
                 t_request,
                 (s->logs.t_queue >= 0) ? s->logs.t_queue - t_request : -1,