Re: [PATCH/RFC] Bugfix for AIX tmpfile creation
- From
Linus Torvalds <torvalds@linux-foundation.org>
- Date
- Jun 23, 2008, 22:55 UTC
- Message-ID
- <alpine.LFD.1.10.0806231552510.2926@woody.linux-foundation.org>
- In-Reply-To
- <7vy74vg4zq.fsf@gitster.siamese.dyndns.org>
On Mon, 23 Jun 2008, Junio C Hamano wrote:
> > Hmm, we have quite a few calls to mkstemp(), so adding the workaround to > this specific call site does not scale well as a fix, I am afraid.
All other callers of mkstemp() always rewrite the _entire_ path and don't care about the result after failure (since it failed)
The new 'create_tmpfile()' usage was/is special in that it re-uses the path without recreating it all, just the final part.
So I think Patrick's patch is fine and sufficient. Maybe I missed some in my quick grep, but it does look ok, and the create_tmpfile() usage really was pretty special.
Linus