tags 307189 +pending thanks This one time, at band camp, Marc Sherman said: > In 302254 you said the "Signal 1: re-opening log file" message was > going away, but it's still here, just without the "Signal 1" part. So > here's a modification of the logcheck line to catch it:
Yeah, it's a little complicated, but the problem was that the logg()
function to write the loglines was called from within a signal handler,
and it would get stuck in a futex_wait() sometimes. I had originally
suggested that they just pull the logg() call altogether in order to fix
this. The next CVS pull I made showed that they did just remove this
line. Subsequently, they decided to do the slightly better fix, which
was to move the logg() call outside of the signal handler, resulting in
where we are now. Sorry about that.
I also don't use syslog with clam, so I miss a lot of the stuff you
report :)
> ^\w{3} [ :0-9]{11} [._[:alnum:]-]+ freshclam\[[0-9]+\]: Received signal:
> (wake up|re-opening log file)$
Will put in for -3.
> By the way, when I wrote the log line for new cvd files in freshclam,
> I had the version as a literal 4 explicitly because the cvd version is
> tied to the clamav version; my intention was to not hide any cvd
> downloads that arrive with a newer version, that would be unsupported.
> I suspect that clamav probably logs a whole bunch of other more
> aggressive sounding messages if that happens, so it's up to you if you
> want to keep it with the permissive [0-9]+ for the version number.
Yes, you'll get the usual 'newer version available, blah, blah' messages
as well, so there's no particular reason to see this one line. Or at
least that was my feeling - if you disagree, I can change it back. It
just seems like losing maintainability for little gain.
Just as a side note - I am going to hold off uploading -3 for a few days
to let -2 get into sarge. I would really like to see sarge get some
version of 0.84 to fix that digest handling bug.
Thanks,
--
-----------------------------------------------------------------
| ,''`. Stephen Gran |
| : :' : [EMAIL PROTECTED] |
| `. `' Debian user, admin, and developer |
| `- http://www.debian.org |
-----------------------------------------------------------------
pgpv1wPcwM2mn.pgp
Description: PGP signature
