[Bug 3651] [review] Fix "oops! still tied to bayes DBs, untie'ing" with compile_now

[bu...@bugzilla.spamassassin.org]

http://bugzilla.spamassassin.org/show_bug.cgi?id=3651

felicity@kluge.net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[patch] Fix "oops! still    |[review] Fix "oops! still
                   |tied to bayes DBs,          |tied to bayes DBs,
                   |untie'ing" with compile_now |untie'ing" with compile_now



------- Additional Comments From felicity@kluge.net  2004-07-31 15:11 -------
the reason we don't run into this, btw, is that we only call compile_now from 2 places: mass-check and 
spamd.  spamd runs it without a bayes db (/tmp/spamd-init-*), so there'll never be anything tied.  it is 
reproducable via mass-check (I'm surprised no one spotted this before!)

currently, finish_learner simply calls untie_db and that's it.  as long as that's all that occurs, I believe the 
patch is the correct fix.  +1 on that.  I'm changing this to review status.



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