]> git.rkrishnan.org Git - tahoe-lafs/tahoe-lafs.git/commit
Fixes fd leak in leasedb (ticket #2015)
authorMark Berger <mark.berger.j@gmail.com>
Fri, 12 Jul 2013 15:09:04 +0000 (11:09 -0400)
committerDaira Hopwood <daira@jacaranda.org>
Fri, 17 Apr 2015 21:31:40 +0000 (22:31 +0100)
commitdcd0e9aad9fd1bebb4fd8e7fe319bcd828f7470a
tree561612377ee21d6d3af02e7411df42c6bdf6a366
parentdb139b584d0a01aa2535d75e76ba2f7fc272f09b
Fixes fd leak in leasedb (ticket #2015)

Brian correctly diagnozed this issue and suggested the fix in Weekly Dev
Chat of 2013-07-09. This patch has been manually tested by Zooko using his
fdleakfinder and a unit test has been added to ensure that database connections
are properly closed. Without this patch, the 1819-cloud-merge branch uses
more than 1020 fds and will start failing on operating systems with a low fds
limit.

The portions affecting leasedb.py were written by Zooko and Daira.
src/allmydata/storage/accountant.py
src/allmydata/storage/leasedb.py
src/allmydata/test/test_leasedb.py