| Summary: | app-antivirus/clamav init script incorrectly fixes log permissions | ||
|---|---|---|---|
| Product: | Gentoo Linux | Reporter: | Richard Freeman <rich0> |
| Component: | Current packages | Assignee: | Antivirus Team <antivirus> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | net-mail+disabled |
| Priority: | Normal | ||
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Package list: | Runtime testing required: | --- | |
+*clamav-0.97.7-r1 (25 Mar 2013) + + 25 Mar 2013; Eray Aslan <eras@gentoo.org> +clamav-0.97.7-r1.ebuild, + +files/clamd.initd-r4: + Fix log file permissions - bug #463208 + This actually still broken. The logrotate calls logfix() directly, which means the user variables are never populated. checkpath defaults to root:root since the arg is empty. I propose moving the variables to the top of the file. That way, both start() and logfix() can use them when called independently. Ah, that doesn't work due to the custom get_config(). I suppose reverting start to the old local ... calls and adding them to logfix() is the Right Thing, but I'll let you decide. Should be fixed in CVS. Please let me know if you still have a problem. Thanks. |
/etc/init.d/clamd logfix is used to fix log permissions after log rotation. It references ${clamd_user} and ${freshclam_user}, but does not define these variables. The code to define them is in the start() function, which is not called during log rotation. The result is an error when freshclam is run from cron: ERROR: Can't open /var/log/clamav/freshclam.log in append mode (check permissions!). ERROR: Problem with internal logger (UpdateLogFile = /var/log/clamav/freshclam.log). Setting these variables within logfix() has been tested and fixes the problem.