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

[EMAIL PROTECTED] changed:

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



------- Additional Comments From [EMAIL PROTECTED]  2004-04-27 13:29 -------
yep.  copy_config()...

fixed in r10323.

commit comment:
bug 3317: copy_config() needs to deal with the one-off object copying (in this 
case the trusted 
networks object array) _before_ dealing with standard reference copying.


I didn't put it in the comment, but the problem was that we'd make a note in 
the backup copy that 
trusted_networks (or internal_networks) should be dealt with, but then when we 
copied from backup to 
the main config we weren't dealing with a NetSet object, just a HASH reference, 
so the conf NetSet 
object would get blown away, and it would be replaced with a HASH reference, 
which has no code 
associated with it.

I moved the generic copy to the bottom of the loop and put in an obvious 
all-uppercase note to leave 
the generic copy at the bottom for this very reason.



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

Reply via email to