mutable: remove some dead code, rearrange use of populate_pubkey

diff --git a/src/allmydata/mutable/node.py b/src/allmydata/mutable/node.py
index 63f83e4f2af319e11e78e836f7f72317c2c5b26d..749b8483e1d7960487aa048208943c607a8229ae 100644 (file)
--- a/src/allmydata/mutable/node.py
+++ b/src/allmydata/mutable/node.py
@@ -137,12 +137,6 @@ class MutableFileNode:
         privkey = enc.process(enc_privkey)
         return privkey
 
-    def _populate(self, stuff):
-        # the Retrieval object calls this with values it discovers when
-        # downloading the slot. This is how a MutableFileNode that was
-        # created from a URI learns about its full key.
-        pass
-
     def _populate_pubkey(self, pubkey):
         self._pubkey = pubkey
     def _populate_required_shares(self, required_shares):

diff --git a/src/allmydata/mutable/servermap.py b/src/allmydata/mutable/servermap.py
index 5d90501a15dfbce8270db4cde8708417a3669984..ea9de5338d1e884142ee25ae3b328c556e9a9b4e 100644 (file)
--- a/src/allmydata/mutable/servermap.py
+++ b/src/allmydata/mutable/servermap.py
@@ -558,13 +558,13 @@ class ServermapUpdater:
         (seqnum, root_hash, IV, k, N, segsize, datalength,
          pubkey_s, signature, prefix) = unpack_prefix_and_signature(data)
 
-        if not self._node._pubkey:
+        if not self._node.get_pubkey():
             fingerprint = hashutil.ssk_pubkey_fingerprint_hash(pubkey_s)
             assert len(fingerprint) == 32
             if fingerprint != self._node._fingerprint:
                 raise CorruptShareError(peerid, shnum,
                                         "pubkey doesn't match fingerprint")
-            self._node._pubkey = self._deserialize_pubkey(pubkey_s)
+            self._node._populate_pubkey(self._deserialize_pubkey(pubkey_s))
 
         if self._need_privkey:
             self._try_to_extract_privkey(data, peerid, shnum)