From: Zooko O'Whielacronx 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/%5B/%5D%20/uri/?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.