You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by jm...@apache.org on 2007/09/27 14:16:32 UTC
svn commit: r579983 - /spamassassin/trunk/spamd/spamd.raw
Author: jm
Date: Thu Sep 27 05:16:31 2007
New Revision: 579983
URL: http://svn.apache.org/viewvc?rev=579983&view=rev
Log:
bug 5606: too-early init_learner() call causes root's user prefs file to be read when spamd is started; this is inappropriate. fix
Modified:
spamassassin/trunk/spamd/spamd.raw
Modified: spamassassin/trunk/spamd/spamd.raw
URL: http://svn.apache.org/viewvc/spamassassin/trunk/spamd/spamd.raw?rev=579983&r1=579982&r2=579983&view=diff
==============================================================================
--- spamassassin/trunk/spamd/spamd.raw (original)
+++ spamassassin/trunk/spamd/spamd.raw Thu Sep 27 05:16:31 2007
@@ -875,10 +875,6 @@
}
);
-$spamtest->init_learner({
- opportunistic_expire_check_only => 1,
-});
-
# if $clients_per_child == 1, there's no point in copying configs around
unless ($clients_per_child > 1) {
# unset $copy_config_p so we don't bother trying to copy things back
@@ -899,6 +895,11 @@
# should be done post-daemonize such that any files created by this
# process are written with the right ownership and everything.
preload_modules_with_tmp_homedir();
+
+# this must be after preload_modules_with_tmp_homedir(), for bug 5606
+$spamtest->init_learner({
+ opportunistic_expire_check_only => 1,
+});
# bayes DBs may still be tied() at this point, so untie them and such.
$spamtest->finish_learner();