ServerMap.copy(): deepcopy .update_data too. Closes #1785.

This was apparently unused, but good to copy for completeness. Tests
pass with and without the change.

diff --git a/src/allmydata/mutable/servermap.py b/src/allmydata/mutable/servermap.py
index 4ef85c583f1f60d86ba6610b405346cb49e6bd26..149e1a259e054adc9c068f36ef917ba221d32590 100644 (file)
--- a/src/allmydata/mutable/servermap.py
+++ b/src/allmydata/mutable/servermap.py
@@ -1,5 +1,5 @@
 
-import sys, time
+import sys, time, copy
 from zope.interface import implements
 from itertools import count
 from twisted.internet import defer
@@ -134,6 +134,7 @@ class ServerMap:
         s._bad_shares = self._bad_shares.copy() # tuple->str
         s._last_update_mode = self._last_update_mode
         s._last_update_time = self._last_update_time
+        s.update_data = copy.deepcopy(self.update_data)
         return s
 
     def get_reachable_servers(self):