From: Christian Göttsche Date: Sat, 23 Jul 2022 14:38:23 +0000 (+0200) Subject: Fix uninitialized read in doveadm-oldstats X-Git-Tag: archive/raspbian/1%2.3.21.1+dfsg1-1+rpi1^2~4 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=5779b73fe3c71dc70484d15eae97b1be19fab561;p=dovecot.git Fix uninitialized read in doveadm-oldstats The third argument to doveadm_cmd_param_bool() is only set on a return value of TRUE. Since disk_input_field and disk_output_field should be set if the value of show-disk-io is specified and specified to true, fix the condition. doveadm-oldstats.c: In function 'cmd_stats_top': doveadm-oldstats.c:551:63: warning: 'b' may be used uninitialized [-Wmaybe-uninitialized] 551 | if (!doveadm_cmd_param_bool(cctx, "show-disk-io", &b) && b) { | ^ doveadm-oldstats.c:545:14: note: 'b' was declared here 545 | bool b; | ^ Gbp-Pq: Name Fix-uninitialized-read-in-doveadm-oldstats.patch --- diff --git a/src/doveadm/doveadm-oldstats.c b/src/doveadm/doveadm-oldstats.c index 4be575e..ff6dcf5 100644 --- a/src/doveadm/doveadm-oldstats.c +++ b/src/doveadm/doveadm-oldstats.c @@ -548,7 +548,7 @@ static void cmd_stats_top(struct doveadm_cmd_context *cctx) path = t_strconcat(doveadm_settings->base_dir, "/old-stats", NULL); } - if (!doveadm_cmd_param_bool(cctx, "show-disk-io", &b) && b) { + if (doveadm_cmd_param_bool(cctx, "show-disk-io", &b) && b) { disk_input_field = "read_bytes"; disk_output_field = "write_bytes"; }