diff options
author | Doug Barton <dougb@FreeBSD.org> | 2011-11-13 03:01:58 +0000 |
---|---|---|
committer | Doug Barton <dougb@FreeBSD.org> | 2011-11-13 03:01:58 +0000 |
commit | 398955cd68c179bd0086150da3bd48f3df369b98 (patch) | |
tree | 69a6cf4d52fd346ebadca1fb48e60a606bd6293b /etc | |
parent | 08907004c697febea93381859c93735425da8d06 (diff) | |
download | src-398955cd68c179bd0086150da3bd48f3df369b98.tar.gz src-398955cd68c179bd0086150da3bd48f3df369b98.zip |
The default setting, daily_accounting_compress="NO", was causing
only 1 old file to be saved, so fix this. Problem raised in the PR,
but actually required a different solution.
While I'm here, fix a very old off-by-one error causing 1 more file
than specified in daily_accounting_save to be saved because acct.0
was not taken into account (pun intended). Change that, and use a more
thorough method of finding old files to delete. Partly just because this
is the right thing to do, but also to silently fix the extra log that
would have been left behind forever with the previous method.
PR: conf/160848
Submitted by: Andrey Zonov <andrey@zonov.org>
Notes
Notes:
svn path=/head/; revision=227482
Diffstat (limited to 'etc')
-rwxr-xr-x | etc/periodic/daily/310.accounting | 18 |
1 files changed, 12 insertions, 6 deletions
diff --git a/etc/periodic/daily/310.accounting b/etc/periodic/daily/310.accounting index b964616c320b..d11745de6eab 100755 --- a/etc/periodic/daily/310.accounting +++ b/etc/periodic/daily/310.accounting @@ -30,8 +30,13 @@ case "$daily_accounting_enable" in cd /var/account rc=0 - n=$daily_accounting_save - rm -f acct.$n.gz acct.$n || rc=3 + n=$(( $daily_accounting_save - 1 )) + for f in acct.*; do + case "$f" in acct.\*) continue ;; esac # No files match + m=${f%.gz} ; m=${m#acct.} + [ $m -ge $n ] && { rm $f || rc=3; } + done + m=$n n=$(($n - 1)) while [ $n -ge 0 ] @@ -44,13 +49,14 @@ case "$daily_accounting_enable" in /etc/rc.d/accounting rotate_log || rc=3 + rm -f acct.merge && cp acct.0 acct.merge || rc=3 + sa -s $daily_accounting_flags /var/account/acct.merge || rc=3 + rm acct.merge + case "$daily_accounting_compress" in [Yy][Ee][Ss]) - gzip --keep -f acct.0 || rc=3;; + gzip -f acct.0 || rc=3;; esac - - sa -s $daily_accounting_flags /var/account/acct.0 && - unlink acct.0 || rc=3 fi;; *) rc=0;; |