Typo in exim filter script

Bug #425206 reported by hackel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
logwatch (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Binary package hint: logwatch

In /usr/share/logwatch/scripts/services/exim on line 60:

$LvlRestarts = $ENV{'exim_restart'} || 5;

However, the variable is referred to later without the trailing "s", starting on line 269:

if ($Detail >= $LvlRestart) {

As a result, the exim queue section is included at any detail level, when it should be omitted at detail level 4 and lower.

Revision history for this message
Chuck Short (zulcss) wrote :

Hi,

Which version of logwatch are you using?

Thanks
chuck

Changed in logwatch (Ubuntu):
status: New → Incomplete
Revision history for this message
Thierry Carrez (ttx) wrote :

Confirmed in current logwatch_7.3.6.cvs20080702-2ubuntu1

Changed in logwatch (Ubuntu):
importance: Undecided → Low
status: Incomplete → Triaged
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package logwatch - 7.3.6.cvs20090906-1ubuntu1

---------------
logwatch (7.3.6.cvs20090906-1ubuntu1) karmic; urgency=low

  * Merge from debian unstable (LP: #228917, #391077, #425206, #443252),
    remaining changes:
    - Use postfix rather than exim4.

logwatch (7.3.6.cvs20090906-1) unstable; urgency=low

  * New CVS snapshot + postfix-logwatch 1.38.01
    - postfix-logwatch now supports SPF \S+ lines (closes: #507937)
  * Support cron with -L2 loglevel (closes: #542453)
  * Move logfiles ending with *.gz or *.bz2 to archive list, so they are
    unpacked before being processed (closes: #536472)
  * Include patch for imapd to support loglines from uw-imapd, submitted by
    Marcin Szewczyk, closes: #541152
  * scripts/services/denyhosts: Output one host per line, not one possibly
    very long line with all hosts. closes: #507042
  * Support "command continues" lines in sudo. closes: #505432
  * Fix typos in exim script. LP: #425206.
  * No longer force recipient in the cron script. (--output mail instead of
    --mailto root)
  * change comment concerning the Mailto parameter to no longer talk about
    removed Print parameter. closes: #499109
  * lintian:
    - Standards-Version: 3.8.3 (no changes)
    - Change path to GPL2 in debian/copyright

 -- Kees Cook <email address hidden> Mon, 05 Oct 2009 09:20:31 -0700

Changed in logwatch (Ubuntu):
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.