From: Zooko O'Whielacronx <zooko@zooko.com>
Date: Fri, 29 Dec 2006 19:40:10 +0000 (-0700)
Subject: encode_new: fix a bug in subshare hashes and tidy-up a couple of things
X-Git-Tag: tahoe_v0.1.0-0-UNSTABLE~419
X-Git-Url: https://git.rkrishnan.org/pf/content/en/service/rgr-080307.php?a=commitdiff_plain;h=dcfddcac41a04cc88fe9c5e30a0111f432b6053c;p=tahoe-lafs%2Ftahoe-lafs.git

encode_new: fix a bug in subshare hashes and tidy-up a couple of things
---

diff --git a/src/allmydata/encode_new.py b/src/allmydata/encode_new.py
index 4e13b21c..50a4f69b 100644
--- a/src/allmydata/encode_new.py
+++ b/src/allmydata/encode_new.py
@@ -43,10 +43,10 @@ tree is sent to the shareholder after all the data has been sent. At
 retrieval time, the decoder will ask for specific pieces of this tree before
 asking for subshares, whichever it needs to validate those subshares.
 
-[TODO: we don't really need to generate this whole subshare hash tree
+(Note: we don't really need to generate this whole subshare hash tree
 ourselves. It would be sufficient to have the shareholder generate it and
 just tell us the root. This gives us an extra level of validation on the
-transfer, though, and it is relatively cheap to compute.]
+transfer, though, and it is relatively cheap to compute.)
 
 Each of these subshare hash trees has a root hash. The collection of these
 root hashes for all shares are collected into the 'share hash tree', which
@@ -72,11 +72,12 @@ class Encoder(object):
         self.num_segments = int(math.ceil(fsize / self.segment_size))
 
         self.num_shares = 100
-        self.share_size = self.file_size / 25
+        self.required_shares = 25
+        self.share_size = self.file_size / self.required_shares
 
     def get_reservation_size(self):
         self.num_shares = 100
-        self.share_size = self.file_size / 25
+        self.share_size = self.file_size / self.required_shares
         overhead = self.compute_overhead()
         return self.share_size + overhead
 
@@ -85,7 +86,7 @@ class Encoder(object):
         self.cryptor = AES.new(key=self.key, mode=AES.MODE_CTR,
                                counterstart="\x00"*16)
         self.segment_num = 0
-        self.subshare_hashes = [[]] * self.num_shares
+        self.subshare_hashes = [[] for x in range(self.num_shares)]
         # subshare_hashes[i] is a list that will be accumulated and then send
         # to landlord[i]. This list contains a hash of each segment_share
         # that we sent to that landlord.