On Mon, 26 Oct 2015 09:47:59 -0400, "Ted Unangst" wrote: > Simplest fix is to just leave the database closed. The point of the cache is > to avoid calling getpwuid() at all, so we shouldn't worry about the > performance of that call so much. Now, the cache is rather stupid, and we can > fix that too, but first fix the real bug.
I don't think we need to change this, the .db fds are already opened with O_CLOEXEC. Keeping the db file open is useful when you are doing lots of lookups, which is probably the case when you are priming the cache. - todd
