Bug#819572: logwatch: redundant argument in sprintf at .../amavis line 1338, <> line 321 message

[Willi Mann]

Hi,

thanks for your fast reply.

Could you try attached patch and let me know whether it works for you?

In case you don't know how to apply the patch (as root):

patch /usr/share/logwatch/scripts/services/amavis < /tmp/amavis.patch

thanks
Willi

--- /usr/share/logwatch/scripts/services/amavis	2016-03-30 23:32:34.000000000 +0200
+++ amavis	2016-07-27 20:53:23.913506745 +0200
@@ -1334,8 +1334,11 @@
                      $$divisor == $Totals{$keyname} ? 100.00 : $Totals{$keyname} * 100 / $$divisor;
             }
             else {
-               push @{$lines[$cur_level]}, 
-                  sprintf "$fmt  %-23s $extra\n", $total, $desc, commify ($Totals{$keyname});
+	       my $out = sprintf "$fmt  %-23s", $total, $desc;
+	       if ($extra) {
+	          $out .= sprintf " $extra", commify ($Totals{$keyname});
+	       }
+               push @{$lines[$cur_level]}, $out;
             }
          }
       }