[Tiger-devel] [PATCH] Oops, I broke the check_logfiles module for Linux.

[Ryan Bradetich]

Hello guys,

The patch I submitted here:
http://mail.nongnu.org/archive/html/tiger-devel/2003-10/txt00011.txt

broke the check_logfiles check for Linux.  The problem is that Linux
does not define the $BTMP variable (something I will address at the end
of this email), so the the following statement hangs:

        group=`getpermit $BTMP | $AWK '{ print $3 }'`

This patch fixes this problem by wrapping the whoe BTMP check with the
$BTMP length check.  [Note: I also took care of this for the LOGINLOG,
and MESSAGES too.]

Now to address the btmp issue... BTMP should be defined for linux,
though it is optional.  Here is the excerpt from the lastb manpage.
[Note: I have been utalizing the btmp file on linux for quite a while,
so I know it works :)].

        The files wtmp and btmp might not be found. The system only logs
        information in these files if  they are present. This is a local
        configuration issue. If you want the files to be used, they can         
be
created with a simple touch(1) command (for example, touch
        /var/log/wtmp).

This patch fixes the BTMP entry for Linux as well.

Thanks,

- Ryan

check_logfiles.diff
Description: Text Data