~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/check.py

  • Committer: Marius Kruger
  • Date: 2010-07-10 21:28:56 UTC
  • mto: (5384.1.1 integration)
  • mto: This revision was merged to the branch mainline in revision 5385.
  • Revision ID: marius.kruger@enerweb.co.za-20100710212856-uq4ji3go0u5se7hx
* Update documentation
* add NEWS

Show diffs side-by-side

added added

removed removed

Lines of Context:
12
12
#
13
13
# You should have received a copy of the GNU General Public License
14
14
# along with this program; if not, write to the Free Software
15
 
# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
16
 
17
17
# TODO: Check ancestries are correct for every revision: includes
18
18
# every committed so far, and in a reasonable order.
21
21
 
22
22
# TODO: Check for extra files in the control directory.
23
23
 
24
 
# TODO: Check revision, inventory and entry objects have all 
 
24
# TODO: Check revision, inventory and entry objects have all
25
25
# required fields.
26
26
 
27
27
# TODO: Get every revision in the revision-store even if they're not
32
32
# raising them.  If there's more than one exception it'd be good to see them
33
33
# all.
34
34
 
35
 
from bzrlib.errors import BzrCheckError
 
35
"""Checking of bzr objects.
 
36
 
 
37
check_refs is a concept used for optimising check. Objects that depend on other
 
38
objects (e.g. tree on repository) can list the objects they would be requesting
 
39
so that when the dependent object is checked, matches can be pulled out and
 
40
evaluated in-line rather than re-reading the same data many times.
 
41
check_refs are tuples (kind, value). Currently defined kinds are:
 
42
* 'trees', where value is a revid and the looked up objects are revision trees.
 
43
* 'lefthand-distance', where value is a revid and the looked up objects are the
 
44
  distance along the lefthand path to NULL for that revid.
 
45
* 'revision-existence', where value is a revid, and the result is True or False
 
46
  indicating that the revision was found/not found.
 
47
"""
 
48
 
 
49
from bzrlib import errors
 
50
from bzrlib.branch import Branch
 
51
from bzrlib.bzrdir import BzrDir
 
52
from bzrlib.revision import NULL_REVISION
 
53
from bzrlib.symbol_versioning import deprecated_function, deprecated_in
 
54
from bzrlib.trace import note
36
55
import bzrlib.ui
37
 
from bzrlib.trace import note
 
56
from bzrlib.workingtree import WorkingTree
38
57
 
39
58
class Check(object):
40
59
    """Check a repository"""
41
60
 
42
61
    # The Check object interacts with InventoryEntry.check, etc.
43
62
 
44
 
    def __init__(self, repository):
 
63
    def __init__(self, repository, check_repo=True):
45
64
        self.repository = repository
46
 
        self.checked_text_cnt = 0
47
65
        self.checked_rev_cnt = 0
48
 
        self.ghosts = []
49
 
        self.repeated_text_cnt = 0
 
66
        self.ghosts = set()
50
67
        self.missing_parent_links = {}
51
68
        self.missing_inventory_sha_cnt = 0
52
69
        self.missing_revision_cnt = 0
53
 
        # maps (file-id, version) -> sha1; used by InventoryFile._check
54
 
        self.checked_texts = {}
55
 
        self.checked_weaves = {}
 
70
        self.checked_weaves = set()
 
71
        self.unreferenced_versions = set()
 
72
        self.inconsistent_parents = []
 
73
        self.rich_roots = repository.supports_rich_root()
 
74
        self.text_key_references = {}
 
75
        self.check_repo = check_repo
 
76
        self.other_results = []
 
77
        # Plain text lines to include in the report
 
78
        self._report_items = []
 
79
        # Keys we are looking for; may be large and need spilling to disk.
 
80
        # key->(type(revision/inventory/text/signature/map), sha1, first-referer)
 
81
        self.pending_keys = {}
 
82
        # Ancestors map for all of revisions being checked; while large helper
 
83
        # functions we call would create it anyway, so better to have once and
 
84
        # keep.
 
85
        self.ancestors = {}
56
86
 
57
 
    def check(self):
 
87
    def check(self, callback_refs=None, check_repo=True):
 
88
        if callback_refs is None:
 
89
            callback_refs = {}
58
90
        self.repository.lock_read()
59
91
        self.progress = bzrlib.ui.ui_factory.nested_progress_bar()
60
92
        try:
61
 
            self.progress.update('retrieving inventory', 0, 0)
62
 
            # do not put in init, as it should be done with progess,
63
 
            # and inside the lock.
64
 
            self.inventory_weave = self.repository.get_inventory_weave()
65
 
            self.plan_revisions()
66
 
            revno = 0
67
 
            self.check_weaves()
68
 
            while revno < len(self.planned_revisions):
69
 
                rev_id = self.planned_revisions[revno]
70
 
                self.progress.update('checking revision', revno,
71
 
                                     len(self.planned_revisions))
72
 
                revno += 1
73
 
                self.check_one_rev(rev_id)
 
93
            self.progress.update('check', 0, 4)
 
94
            if self.check_repo:
 
95
                self.progress.update('checking revisions', 0)
 
96
                self.check_revisions()
 
97
                self.progress.update('checking commit contents', 1)
 
98
                self.repository._check_inventories(self)
 
99
                self.progress.update('checking file graphs', 2)
 
100
                # check_weaves is done after the revision scan so that
 
101
                # revision index is known to be valid.
 
102
                self.check_weaves()
 
103
            self.progress.update('checking branches and trees', 3)
 
104
            if callback_refs:
 
105
                repo = self.repository
 
106
                # calculate all refs, and callback the objects requesting them.
 
107
                refs = {}
 
108
                wanting_items = set()
 
109
                # Current crude version calculates everything and calls
 
110
                # everything at once. Doing a queue and popping as things are
 
111
                # satisfied would be cheaper on memory [but few people have
 
112
                # huge numbers of working trees today. TODO: fix before
 
113
                # landing].
 
114
                distances = set()
 
115
                existences = set()
 
116
                for ref, wantlist in callback_refs.iteritems():
 
117
                    wanting_items.update(wantlist)
 
118
                    kind, value = ref
 
119
                    if kind == 'trees':
 
120
                        refs[ref] = repo.revision_tree(value)
 
121
                    elif kind == 'lefthand-distance':
 
122
                        distances.add(value)
 
123
                    elif kind == 'revision-existence':
 
124
                        existences.add(value)
 
125
                    else:
 
126
                        raise AssertionError(
 
127
                            'unknown ref kind for ref %s' % ref)
 
128
                node_distances = repo.get_graph().find_lefthand_distances(distances)
 
129
                for key, distance in node_distances.iteritems():
 
130
                    refs[('lefthand-distance', key)] = distance
 
131
                    if key in existences and distance > 0:
 
132
                        refs[('revision-existence', key)] = True
 
133
                        existences.remove(key)
 
134
                parent_map = repo.get_graph().get_parent_map(existences)
 
135
                for key in parent_map:
 
136
                    refs[('revision-existence', key)] = True
 
137
                    existences.remove(key)
 
138
                for key in existences:
 
139
                    refs[('revision-existence', key)] = False
 
140
                for item in wanting_items:
 
141
                    if isinstance(item, WorkingTree):
 
142
                        item._check(refs)
 
143
                    if isinstance(item, Branch):
 
144
                        self.other_results.append(item.check(refs))
74
145
        finally:
75
146
            self.progress.finished()
76
147
            self.repository.unlock()
77
148
 
78
 
    def plan_revisions(self):
79
 
        repository = self.repository
80
 
        self.planned_revisions = set(repository.all_revision_ids())
81
 
        self.progress.clear()
82
 
        inventoried = set(self.inventory_weave.versions())
83
 
        awol = self.planned_revisions - inventoried
84
 
        if len(awol) > 0:
85
 
            raise BzrCheckError('Stored revisions missing from inventory'
86
 
                '{%s}' % ','.join([f for f in awol]))
 
149
    def _check_revisions(self, revisions_iterator):
 
150
        """Check revision objects by decorating a generator.
 
151
 
 
152
        :param revisions_iterator: An iterator of(revid, Revision-or-None).
 
153
        :return: A generator of the contents of revisions_iterator.
 
154
        """
 
155
        self.planned_revisions = set()
 
156
        for revid, revision in revisions_iterator:
 
157
            yield revid, revision
 
158
            self._check_one_rev(revid, revision)
 
159
        # Flatten the revisions we found to guarantee consistent later
 
160
        # iteration.
87
161
        self.planned_revisions = list(self.planned_revisions)
 
162
        # TODO: extract digital signatures as items to callback on too.
 
163
 
 
164
    def check_revisions(self):
 
165
        """Scan revisions, checking data directly available as we go."""
 
166
        revision_iterator = self.repository._iter_revisions(None)
 
167
        revision_iterator = self._check_revisions(revision_iterator)
 
168
        # We read the all revisions here:
 
169
        # - doing this allows later code to depend on the revision index.
 
170
        # - we can fill out existence flags at this point
 
171
        # - we can read the revision inventory sha at this point
 
172
        # - we can check properties and serialisers etc.
 
173
        if not self.repository.revision_graph_can_have_wrong_parents():
 
174
            # The check against the index isn't needed.
 
175
            self.revs_with_bad_parents_in_index = None
 
176
            for thing in revision_iterator:
 
177
                pass
 
178
        else:
 
179
            bad_revisions = self.repository._find_inconsistent_revision_parents(
 
180
                revision_iterator)
 
181
            self.revs_with_bad_parents_in_index = list(bad_revisions)
88
182
 
89
183
    def report_results(self, verbose):
 
184
        if self.check_repo:
 
185
            self._report_repo_results(verbose)
 
186
        for result in self.other_results:
 
187
            result.report_results(verbose)
 
188
 
 
189
    def _report_repo_results(self, verbose):
90
190
        note('checked repository %s format %s',
91
 
             self.repository.bzrdir.root_transport,
92
 
             self.repository._format)
 
191
            self.repository.user_url,
 
192
            self.repository._format)
93
193
        note('%6d revisions', self.checked_rev_cnt)
94
 
        note('%6d unique file texts', self.checked_text_cnt)
95
 
        note('%6d repeated file texts', self.repeated_text_cnt)
96
 
        note('%6d weaves', len(self.checked_weaves))
 
194
        note('%6d file-ids', len(self.checked_weaves))
 
195
        if verbose:
 
196
            note('%6d unreferenced text versions',
 
197
                len(self.unreferenced_versions))
 
198
        if verbose and len(self.unreferenced_versions):
 
199
                for file_id, revision_id in self.unreferenced_versions:
 
200
                    note('unreferenced version: {%s} in %s', revision_id,
 
201
                        file_id)
97
202
        if self.missing_inventory_sha_cnt:
98
203
            note('%6d revisions are missing inventory_sha1',
99
204
                 self.missing_inventory_sha_cnt)
113
218
                    note('      %s should be in the ancestry for:', link)
114
219
                    for linker in linkers:
115
220
                        note('       * %s', linker)
116
 
 
117
 
    def check_one_rev(self, rev_id):
118
 
        """Check one revision.
119
 
 
120
 
        rev_id - the one to check
 
221
        if len(self.inconsistent_parents):
 
222
            note('%6d inconsistent parents', len(self.inconsistent_parents))
 
223
            if verbose:
 
224
                for info in self.inconsistent_parents:
 
225
                    revision_id, file_id, found_parents, correct_parents = info
 
226
                    note('      * %s version %s has parents %r '
 
227
                         'but should have %r'
 
228
                         % (file_id, revision_id, found_parents,
 
229
                             correct_parents))
 
230
        if self.revs_with_bad_parents_in_index:
 
231
            note('%6d revisions have incorrect parents in the revision index',
 
232
                 len(self.revs_with_bad_parents_in_index))
 
233
            if verbose:
 
234
                for item in self.revs_with_bad_parents_in_index:
 
235
                    revision_id, index_parents, actual_parents = item
 
236
                    note(
 
237
                        '       %s has wrong parents in index: '
 
238
                        '%r should be %r',
 
239
                        revision_id, index_parents, actual_parents)
 
240
        for item in self._report_items:
 
241
            note(item)
 
242
 
 
243
    def _check_one_rev(self, rev_id, rev):
 
244
        """Cross-check one revision.
 
245
 
 
246
        :param rev_id: A revision id to check.
 
247
        :param rev: A revision or None to indicate a missing revision.
121
248
        """
122
 
        rev = self.repository.get_revision(rev_id)
123
 
                
124
249
        if rev.revision_id != rev_id:
125
 
            raise BzrCheckError('wrong internal revision id in revision {%s}'
126
 
                                % rev_id)
127
 
 
 
250
            self._report_items.append(
 
251
                'Mismatched internal revid {%s} and index revid {%s}' % (
 
252
                rev.revision_id, rev_id))
 
253
            rev_id = rev.revision_id
 
254
        # Check this revision tree etc, and count as seen when we encounter a
 
255
        # reference to it.
 
256
        self.planned_revisions.add(rev_id)
 
257
        # It is not a ghost
 
258
        self.ghosts.discard(rev_id)
 
259
        # Count all parents as ghosts if we haven't seen them yet.
128
260
        for parent in rev.parent_ids:
129
261
            if not parent in self.planned_revisions:
130
 
                missing_links = self.missing_parent_links.get(parent, [])
131
 
                missing_links.append(rev_id)
132
 
                self.missing_parent_links[parent] = missing_links
133
 
                # list based so somewhat slow,
134
 
                # TODO have a planned_revisions list and set.
135
 
                if self.repository.has_revision(parent):
136
 
                    missing_ancestry = self.repository.get_ancestry(parent)
137
 
                    for missing in missing_ancestry:
138
 
                        if (missing is not None 
139
 
                            and missing not in self.planned_revisions):
140
 
                            self.planned_revisions.append(missing)
141
 
                else:
142
 
                    self.ghosts.append(rev_id)
143
 
 
144
 
        if rev.inventory_sha1:
145
 
            inv_sha1 = self.repository.get_inventory_sha1(rev_id)
146
 
            if inv_sha1 != rev.inventory_sha1:
147
 
                raise BzrCheckError('Inventory sha1 hash doesn\'t match'
148
 
                    ' value in revision {%s}' % rev_id)
149
 
        self._check_revision_tree(rev_id)
 
262
                self.ghosts.add(parent)
 
263
        
 
264
        self.ancestors[rev_id] = tuple(rev.parent_ids) or (NULL_REVISION,)
 
265
        self.add_pending_item(rev_id, ('inventories', rev_id), 'inventory',
 
266
            rev.inventory_sha1)
150
267
        self.checked_rev_cnt += 1
151
268
 
 
269
    def add_pending_item(self, referer, key, kind, sha1):
 
270
        """Add a reference to a sha1 to be cross checked against a key.
 
271
 
 
272
        :param referer: The referer that expects key to have sha1.
 
273
        :param key: A storage key e.g. ('texts', 'foo@bar-20040504-1234')
 
274
        :param kind: revision/inventory/text/map/signature
 
275
        :param sha1: A hex sha1 or None if no sha1 is known.
 
276
        """
 
277
        existing = self.pending_keys.get(key)
 
278
        if existing:
 
279
            if sha1 != existing[1]:
 
280
                self._report_items.append('Multiple expected sha1s for %s. {%s}'
 
281
                    ' expects {%s}, {%s} expects {%s}', (
 
282
                    key, referer, sha1, existing[1], existing[0]))
 
283
        else:
 
284
            self.pending_keys[key] = (kind, sha1, referer)
 
285
 
152
286
    def check_weaves(self):
153
287
        """Check all the weaves we can get our hands on.
154
288
        """
155
 
        n_weaves = 1
156
289
        weave_ids = []
157
 
        if self.repository.weave_store.listable():
158
 
            weave_ids = list(self.repository.weave_store)
159
 
            n_weaves = len(weave_ids)
160
 
        self.progress.update('checking weave', 0, n_weaves)
161
 
        self.inventory_weave.check(progress_bar=self.progress)
162
 
        for i, weave_id in enumerate(weave_ids):
163
 
            self.progress.update('checking weave', i, n_weaves)
164
 
            w = self.repository.weave_store.get_weave(weave_id,
165
 
                    self.repository.get_transaction())
166
 
            # No progress here, because it looks ugly.
167
 
            w.check()
168
 
            self.checked_weaves[weave_id] = True
169
 
 
170
 
    def _check_revision_tree(self, rev_id):
171
 
        tree = self.repository.revision_tree(rev_id)
172
 
        inv = tree.inventory
173
 
        seen_ids = {}
174
 
        for file_id in inv:
175
 
            if file_id in seen_ids:
176
 
                raise BzrCheckError('duplicated file_id {%s} '
177
 
                                    'in inventory for revision {%s}'
178
 
                                    % (file_id, rev_id))
179
 
            seen_ids[file_id] = True
180
 
        for file_id in inv:
181
 
            ie = inv[file_id]
182
 
            ie.check(self, rev_id, inv, tree)
183
 
        seen_names = {}
184
 
        for path, ie in inv.iter_entries():
185
 
            if path in seen_names:
186
 
                raise BzrCheckError('duplicated path %s '
187
 
                                    'in inventory for revision {%s}'
188
 
                                    % (path, rev_id))
189
 
            seen_names[path] = True
190
 
 
191
 
 
 
290
        storebar = bzrlib.ui.ui_factory.nested_progress_bar()
 
291
        try:
 
292
            self._check_weaves(storebar)
 
293
        finally:
 
294
            storebar.finished()
 
295
 
 
296
    def _check_weaves(self, storebar):
 
297
        storebar.update('text-index', 0, 2)
 
298
        if self.repository._format.fast_deltas:
 
299
            # We haven't considered every fileid instance so far.
 
300
            weave_checker = self.repository._get_versioned_file_checker(
 
301
                ancestors=self.ancestors)
 
302
        else:
 
303
            weave_checker = self.repository._get_versioned_file_checker(
 
304
                text_key_references=self.text_key_references,
 
305
                ancestors=self.ancestors)
 
306
        storebar.update('file-graph', 1)
 
307
        result = weave_checker.check_file_version_parents(
 
308
            self.repository.texts)
 
309
        self.checked_weaves = weave_checker.file_ids
 
310
        bad_parents, unused_versions = result
 
311
        bad_parents = bad_parents.items()
 
312
        for text_key, (stored_parents, correct_parents) in bad_parents:
 
313
            # XXX not ready for id join/split operations.
 
314
            weave_id = text_key[0]
 
315
            revision_id = text_key[-1]
 
316
            weave_parents = tuple([parent[-1] for parent in stored_parents])
 
317
            correct_parents = tuple([parent[-1] for parent in correct_parents])
 
318
            self.inconsistent_parents.append(
 
319
                (revision_id, weave_id, weave_parents, correct_parents))
 
320
        self.unreferenced_versions.update(unused_versions)
 
321
 
 
322
    def _add_entry_to_text_key_references(self, inv, entry):
 
323
        if not self.rich_roots and entry.name == '':
 
324
            return
 
325
        key = (entry.file_id, entry.revision)
 
326
        self.text_key_references.setdefault(key, False)
 
327
        if entry.revision == inv.revision_id:
 
328
            self.text_key_references[key] = True
 
329
 
 
330
 
 
331
@deprecated_function(deprecated_in((1,6,0)))
192
332
def check(branch, verbose):
193
333
    """Run consistency checks on a branch.
194
 
    
195
 
    Results are reported through logging.
196
 
    
 
334
 
 
335
    Results are reported through logging.
 
336
 
 
337
    Deprecated in 1.6.  Please use check_dwim instead.
 
338
 
 
339
    :raise BzrCheckError: if there's a consistency error.
 
340
    """
 
341
    check_branch(branch, verbose)
 
342
 
 
343
 
 
344
@deprecated_function(deprecated_in((1,16,0)))
 
345
def check_branch(branch, verbose):
 
346
    """Run consistency checks on a branch.
 
347
 
 
348
    Results are reported through logging.
 
349
 
197
350
    :raise BzrCheckError: if there's a consistency error.
198
351
    """
199
352
    branch.lock_read()
200
353
    try:
201
 
        branch_result = branch.check()
202
 
        repo_result = branch.repository.check([branch.last_revision()])
 
354
        needed_refs = {}
 
355
        for ref in branch._get_check_refs():
 
356
            needed_refs.setdefault(ref, []).append(branch)
 
357
        result = branch.repository.check([branch.last_revision()], needed_refs)
 
358
        branch_result = result.other_results[0]
203
359
    finally:
204
360
        branch.unlock()
205
361
    branch_result.report_results(verbose)
206
 
    repo_result.report_results(verbose)
 
362
 
 
363
 
 
364
def scan_branch(branch, needed_refs, to_unlock):
 
365
    """Scan a branch for refs.
 
366
 
 
367
    :param branch:  The branch to schedule for checking.
 
368
    :param needed_refs: Refs we are accumulating.
 
369
    :param to_unlock: The unlock list accumulating.
 
370
    """
 
371
    note("Checking branch at '%s'." % (branch.base,))
 
372
    branch.lock_read()
 
373
    to_unlock.append(branch)
 
374
    branch_refs = branch._get_check_refs()
 
375
    for ref in branch_refs:
 
376
        reflist = needed_refs.setdefault(ref, [])
 
377
        reflist.append(branch)
 
378
 
 
379
 
 
380
def scan_tree(base_tree, tree, needed_refs, to_unlock):
 
381
    """Scan a tree for refs.
 
382
 
 
383
    :param base_tree: The original tree check opened, used to detect duplicate
 
384
        tree checks.
 
385
    :param tree:  The tree to schedule for checking.
 
386
    :param needed_refs: Refs we are accumulating.
 
387
    :param to_unlock: The unlock list accumulating.
 
388
    """
 
389
    if base_tree is not None and tree.basedir == base_tree.basedir:
 
390
        return
 
391
    note("Checking working tree at '%s'." % (tree.basedir,))
 
392
    tree.lock_read()
 
393
    to_unlock.append(tree)
 
394
    tree_refs = tree._get_check_refs()
 
395
    for ref in tree_refs:
 
396
        reflist = needed_refs.setdefault(ref, [])
 
397
        reflist.append(tree)
 
398
 
 
399
 
 
400
def check_dwim(path, verbose, do_branch=False, do_repo=False, do_tree=False):
 
401
    """Check multiple objects.
 
402
 
 
403
    If errors occur they are accumulated and reported as far as possible, and
 
404
    an exception raised at the end of the process.
 
405
    """
 
406
    try:
 
407
        base_tree, branch, repo, relpath = \
 
408
                        BzrDir.open_containing_tree_branch_or_repository(path)
 
409
    except errors.NotBranchError:
 
410
        base_tree = branch = repo = None
 
411
 
 
412
    to_unlock = []
 
413
    needed_refs= {}
 
414
    try:
 
415
        if base_tree is not None:
 
416
            # If the tree is a lightweight checkout we won't see it in
 
417
            # repo.find_branches - add now.
 
418
            if do_tree:
 
419
                scan_tree(None, base_tree, needed_refs, to_unlock)
 
420
            branch = base_tree.branch
 
421
        if branch is not None:
 
422
            # We have a branch
 
423
            if repo is None:
 
424
                # The branch is in a shared repository
 
425
                repo = branch.repository
 
426
        if repo is not None:
 
427
            repo.lock_read()
 
428
            to_unlock.append(repo)
 
429
            branches = repo.find_branches(using=True)
 
430
            saw_tree = False
 
431
            if do_branch or do_tree:
 
432
                for branch in branches:
 
433
                    if do_tree:
 
434
                        try:
 
435
                            tree = branch.bzrdir.open_workingtree()
 
436
                            saw_tree = True
 
437
                        except (errors.NotLocalUrl, errors.NoWorkingTree):
 
438
                            pass
 
439
                        else:
 
440
                            scan_tree(base_tree, tree, needed_refs, to_unlock)
 
441
                    if do_branch:
 
442
                        scan_branch(branch, needed_refs, to_unlock)
 
443
            if do_branch and not branches:
 
444
                note("No branch found at specified location.")
 
445
            if do_tree and base_tree is None and not saw_tree:
 
446
                note("No working tree found at specified location.")
 
447
            if do_repo or do_branch or do_tree:
 
448
                if do_repo:
 
449
                    note("Checking repository at '%s'."
 
450
                         % (repo.user_url,))
 
451
                result = repo.check(None, callback_refs=needed_refs,
 
452
                    check_repo=do_repo)
 
453
                result.report_results(verbose)
 
454
        else:
 
455
            if do_tree:
 
456
                note("No working tree found at specified location.")
 
457
            if do_branch:
 
458
                note("No branch found at specified location.")
 
459
            if do_repo:
 
460
                note("No repository found at specified location.")
 
461
    finally:
 
462
        for thing in to_unlock:
 
463
            thing.unlock()