From c7f65ee8ad254f3f84b20706b3f2048c77994785 Mon Sep 17 00:00:00 2001 From: david-sarah Date: Tue, 2 Aug 2011 10:24:37 -0700 Subject: [PATCH] verifier: correct a bug introduced in changeset [5106] that caused us to only verify the first block of a file. refs #1395 --- src/allmydata/immutable/checker.py | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/src/allmydata/immutable/checker.py b/src/allmydata/immutable/checker.py index 6731e94c..ad7931b9 100644 --- a/src/allmydata/immutable/checker.py +++ b/src/allmydata/immutable/checker.py @@ -615,18 +615,21 @@ class Checker(log.PrefixingLogMixin): assert isinstance(r, str), r # to free up the RAM return None + def _get_blocks(vrbp): def _get_block(ign, blocknum): db = vrbp.get_block(blocknum) db.addCallback(_discard_result) return db + dbs = defer.succeed(None) for blocknum in range(veup.num_segments): dbs.addCallback(_get_block, blocknum) - # The Deferred we return will fire after every block of this - # share has been downloaded and verified successfully, or else it - # will errback as soon as the first error is observed. - return dbs + + # The Deferred we return will fire after every block of this + # share has been downloaded and verified successfully, or else it + # will errback as soon as the first error is observed. + return dbs d.addCallback(_get_blocks) -- 2.37.2