_readdir_pyx.so leaks file descriptors

Bug #583486 reported by John A Meinel
This bug affects 3 people
Affects Status Importance Assigned to Milestone
John A Meinel
John A Meinel
John A Meinel
bzr (Ubuntu)

Bug Description

James Troup was kind enough to help track this down on IRC.

this script is enough to trigger the problem:

  rm -fr test; mkdir test; cd test; bzr init
  for i in $(seq 1 1200); do echo a > ${i}; done; bzr add; bzr ci -m "init"
  bzr st # works
  bzr st * # fails

Will trigger it. If I delete "_readdir_pyx.so", then the status is happy.

Marked High but not Critical, because we've had this problem for a while (confirmed in 2.0.4 on Hardy, and 2.1.1 on Lucid, and in bzr.dev on Lucid.)

Related branches

John A Meinel (jameinel)
Changed in bzr:
assignee: nobody → John A Meinel (jameinel)
milestone: none → 2.2b3
status: Confirmed → Fix Released
Revision history for this message
Clint Byrum (clint-fewbar) wrote : Please test proposed package

Accepted bzr into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in bzr (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Jelmer Vernooij (jelmer)
Changed in bzr (Ubuntu):
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package bzr - 2.1.4-0ubuntu1

bzr (2.1.4-0ubuntu1) lucid-proposed; urgency=low

  * Update watch file to use 2.1 series.
  * New upstream release.
   + Fix file descriptors leaks in dirstate compiled extension. LP: #583486
   + Refuse to stack on repositories in incompatible formats. LP: #562380
   + Don't delete nested trees/repos. LP: #572098
   + Fix 'bzr switch' crash when a 'ConfigurableFileMerger' is used. LP: #559436
   + Fix compatibility with older smart servers. LP: #528041
   + Fix symlinks addition. LP: #192859
   + Properly unversion children of unversioned directories. LP: #494221
   + Lock configuration files in '~/.bazaar' for updates. LP: #525571
   + Fix 'bzr commit <symlink>'. LP: #128562
   + Fix `lp:` urls when behind an http proxy. LP: #558343
   + Stop using edge.launchpad.net. LP: #583667
 -- Max Bowsher <email address hidden> Tue, 17 May 2011 09:54:17 +0100

Changed in bzr (Ubuntu Lucid):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers