← Back to branch summary

~bzr-pqm/bzr/bzr.dev

« back to all changes in this revision

Viewing changes to bzrlib/_readdir_pyx.pyx

  • Committer: John Arbash Meinel
  • Date: 2010-05-20 21:23:47 UTC
  • mto: (4634.151.5 2.0)
  • mto: This revision was merged to the branch mainline in revision 5245.
  • Revision ID: john@arbash-meinel.com-20100520212347-y6onphkgkspziji2
https://launchpad.net/bugs/583486
Bug #583486, close the current-dir file descriptor

This code path is triggered when you are given a bunch of files rather than directories
to commands like 'commit' and 'status'. It would try to check for files in each
path passed in, and leak a file descriptor for each one that was not a directory.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/_readdir_pyx.pyx
1
 
# Copyright (C) 2006, 2008, 2009 Canonical Ltd
 
1
# Copyright (C) 2006, 2008, 2009, 2010 Canonical Ltd
2
2
#
3
3
# This program is free software; you can redistribute it and/or modify
4
4
# it under the terms of the GNU General Public License as published by
307
307
        if orig_dir_fd == -1:
308
308
            raise_os_error(errno, "open: ", ".")
309
309
        if -1 == chdir(path):
 
310
            # Ignore the return value, because we are already raising an
 
311
            # exception
 
312
            close(orig_dir_fd)
310
313
            raise_os_error(errno, "chdir: ", path)
311
314
    else:
312
315
        orig_dir_fd = -1