Mailing List Archive

[Bug 3199] Empty $atime in BayesStore.pm:1279 makes noise in syslog
http://bugzilla.spamassassin.org/show_bug.cgi?id=3199





------- Additional Comments From felicity@kluge.net 2004-03-20 09:19 -------
Subject: Re: New: Empty $atime in BayesStore.pm:1279 makes noise in syslog

On Sat, Mar 20, 2004 at 08:37:55AM -0800, bugzilla-daemon@bugzilla.spamassassin.org wrote:
> the time (and using it is a big improvement! :-)). That "fix" is just a hack of
> course, but I've seem it for a long time now, so I'll report it.

The problem, of course, is that $atime _has_ to be set at that point in
the code, so I don't know how it would be "". :(





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 3199] Empty $atime in BayesStore.pm:1279 makes noise in syslog [ In reply to ]
http://bugzilla.spamassassin.org/show_bug.cgi?id=3199

felicity@kluge.net changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |WORKSFORME



------- Additional Comments From felicity@kluge.net 2004-03-23 08:06 -------
I did some more digging. $atime actually doesn't have to be set at that line, but if it's not, there'd be
similar errors from other lines. so as long as there aren't issues from, say, lines 1241 or 1229, atime
_has_ to be a numeric value.

(basically the 1247 error comes from tok_put, which is only called by tok_sync_counters and
tok_touch_token which do numeric less-than/greater-than checks against $atime as well... so if they
don't throw a warning, the value has to be correct for tok_put...)

I'm going to close this as wfm since it's never been reproducable for me. 3.0 has a little more paranoia
going on, so it definitely shouldn't happen there at all.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.