Incorrect generated logrotate file
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ALT Linux |
Invalid
|
Unknown
|
|||
clamav (Ubuntu) |
Fix Released
|
Low
|
Louis Bouchard | ||
Precise |
Fix Released
|
Low
|
Louis Bouchard | ||
Quantal |
Fix Released
|
Low
|
Louis Bouchard | ||
Raring |
Won't Fix
|
Low
|
Louis Bouchard | ||
Saucy |
Fix Released
|
Low
|
Louis Bouchard |
Bug Description
SRU justification :
Without this fix, installation scripts can generate invalid configuration files
(i.e. logrotate.
Impact :
Invalid configuration files
Fix :
Make the parsing algorithm a bit more discriminative
Test Case :
1) install clamav & clamav-daemon
$ sudo apt-get -y install clamav clamav-daemon
2) Add the following line before the first LogFile statement
LogFileMaxSize 0
3) Deinstall without purging the config file
apt-get -y remove clamav clamav-daemon clamav-base clamav-freshclam libclamav6
4) Re-install clamav & clamav-daemon & keep the existing config file
apt-get -y install clamav clamav-daemon clamav-base clamav-freshclam libclamav6
The clamav-daemon file should look like this (which is wrong) :
0 {
rotate 12
weekly
compress
delaycompress
create 640 clamav adm
postrotate
/etc/
endscript
}
The correct format of the file should be :
/var/log/
rotate 12
weekly
compress
delaycompress
create 640 clamav adm
postrotate
/etc/
endscript
}
Regression :
None expected, the values returned are still valid.
Original description of the problem :
Binary package hint: clamav-daemon
Version: 0.96.5+
Sometimes the logrotate conf file (/etc/logrotate
I suspect the root cause is in the clamav-
Cheers,
Javier
Related branches
- Martin Pitt: Approve
-
Diff: 28 lines (+9/-1)2 files modifieddebian/changelog (+8/-0)
debian/common_functions (+1/-1)
Changed in altlinux: | |
status: | Unknown → Invalid |
Changed in clamav (Ubuntu): | |
assignee: | nobody → Louis Bouchard (louis-bouchard) |
Changed in clamav (Ubuntu Precise): | |
assignee: | nobody → Louis Bouchard (louis-bouchard) |
Changed in clamav (Ubuntu Raring): | |
assignee: | nobody → Louis Bouchard (louis-bouchard) |
Changed in clamav (Ubuntu Quantal): | |
assignee: | nobody → Louis Bouchard (louis-bouchard) |
Changed in clamav (Ubuntu Precise): | |
importance: | Undecided → Low |
Changed in clamav (Ubuntu Quantal): | |
importance: | Undecided → Low |
Changed in clamav (Ubuntu Raring): | |
importance: | Undecided → Low |
Changed in clamav (Ubuntu): | |
status: | Confirmed → In Progress |
Changed in clamav (Ubuntu Precise): | |
status: | Confirmed → In Progress |
Changed in clamav (Ubuntu Quantal): | |
status: | Confirmed → In Progress |
Changed in clamav (Ubuntu Raring): | |
status: | Confirmed → In Progress |
Changed in clamav (Ubuntu Saucy): | |
assignee: | nobody → Louis Bouchard (louis-bouchard) |
importance: | Undecided → Low |
status: | New → In Progress |
Thanks for the bug report, Ill see if I can get this fixed.
Regards
chuck