aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJoey Hess <joey@kitenet.net>2008-11-20 13:56:28 -0500
committerJunio C Hamano <gitster@pobox.com>2008-11-23 19:44:19 -0800
commitcbacbf4e55ba53d35567578d8c97fdea6b52e16a (patch)
tree91dec4be3adffad7e665efb5d695ade2a9febd1c
parent3b91c30b763d972c5e133507ccc4b749dc2c6df5 (diff)
downloadgit-cbacbf4e55ba53d35567578d8c97fdea6b52e16a.tar.gz
git-cbacbf4e55ba53d35567578d8c97fdea6b52e16a.tar.xz
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>
-rw-r--r--sha1_file.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sha1_file.c b/sha1_file.c
index 0106e2ce5..36dad7261 100644
--- a/sha1_file.c
+++ b/sha1_file.c
@@ -2289,7 +2289,7 @@ static int create_tmpfile(char *buffer, size_t bufsiz, const char *filename)
memcpy(buffer, filename, dirlen);
strcpy(buffer + dirlen, "tmp_obj_XXXXXX");
fd = mkstemp(buffer);
- if (fd < 0 && dirlen) {
+ if (fd < 0 && dirlen && errno == ENOENT) {
/* Make sure the directory exists */
memcpy(buffer, filename, dirlen);
buffer[dirlen-1] = 0;