Subject: Re: [PATCH 2/2] server-info.c: remove temporary info files on exit

From: Junio C Hamano <[email-masked]>
Date: Fri, 07 Jun 2024 14:49:02 -0700

Taylor Blau <[email-masked]> writes:

>> We no longer do so.  We later call rename_tempfile() to close the
>> underlying file descriptor and move the temporary file to its final
>> place, but I do not see what guarantee we have that we do not lose
>> what we had buffered in the stdio with the updated code.
>
> rename_tempfile() first calls close_tempfile_gently() before calling
> rename(). close_tempfile_gently() calls fclose() on temp->fp before
> returns, so we get our fflush() call implicitly there.

OK.  I didn't see that fclose().  Then we are safe.

Thanks.