From: Johannes Schindelin Date: Fri, 14 Dec 2007 20:15:54 GMT Subject: Re: [PATCH 1/2] Fix config lockfile handling. Message-ID: In-Reply-To: <7vmysdqbui.fsf@gitster.siamese.dyndns.org> Hi, On Fri, 14 Dec 2007, Junio C Hamano wrote: > Johannes Schindelin writes: > > > On Fri, 14 Dec 2007, Kristian Høgsberg wrote: > > > >> - struct lock_file *lock = NULL; > >> + struct lock_file lock; > > > > AFAICT this cannot work. At least not reliably. An atexit() handler > > will access all (even closed) lockfiles. > > Correct. It cannot be on the stack. Note that this behaviour will be another obstacle to libification. Ciao, Dscho