Code

sha1_file: avoid bogus "file exists" error message
authorJoey Hess <joey@kitenet.net>
Thu, 20 Nov 2008 18:56:28 +0000 (13:56 -0500)
committerJunio C Hamano <gitster@pobox.com>
Fri, 28 Nov 2008 02:48:53 +0000 (18:48 -0800)
commit65117abc040d95ef9877c3b14a24f4bc6aeaf4cb
treebcd3e9c86d1bbe25c17723c4fee8ba8959d7f430
parent1510dbe380a24fdf303a3c0594752cfdc968cb12
sha1_file: avoid bogus "file exists" error message

This avoids the following misleading error message:

error: unable to create temporary sha1 filename ./objects/15: File exists

mkstemp can fail for many reasons, one of which, ENOENT, can occur if
the directory for the temp file doesn't exist. create_tmpfile tried to
handle this case by always trying to mkdir the directory, even if it
already existed. This caused errno to be clobbered, so one cannot tell
why mkstemp really failed, and it truncated the buffer to just the
directory name, resulting in the strange error message shown above.

Note that in both occasions that I've seen this failure, it has not been
due to a missing directory, or bad permissions, but some other, unknown
mkstemp failure mode that did not occur when I ran git again. This code
could perhaps be made more robust by retrying mkstemp, in case it was a
transient failure.

Signed-off-by: Joey Hess <joey@kitenet.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
sha1_file.c