aboutsummaryrefslogtreecommitdiff
path: root/path.c
diff options
context:
space:
mode:
authorMatthieu Moy <Matthieu.Moy@imag.fr>2010-02-22 23:32:15 +0100
committerJunio C Hamano <gitster@pobox.com>2010-02-22 15:24:46 -0800
commit1d9740cb324f7f5d798ecfc259dc213b244ad9b7 (patch)
tree1309b7e6d7b38cfc0d5fb0199905eee59d7a4e3c /path.c
parentf80c7ae8fe9c0f3ce93c96a2dccaba34e456e33a (diff)
downloadgit-1d9740cb324f7f5d798ecfc259dc213b244ad9b7.tar.gz
git-1d9740cb324f7f5d798ecfc259dc213b244ad9b7.tar.xz
git_mkstemps_mode: don't set errno to EINVAL on exit.
When reaching the end of git_mkstemps_mode, at least one call to open() has been done, and errno has been set accordingly. Setting errno is therefore not necessary, and actually harmfull since callers can't distinguish e.g. permanent failure from ENOENT, which can just mean that we need to create the containing directory. Signed-off-by: Matthieu Moy <Matthieu.Moy@imag.fr> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'path.c')
-rw-r--r--path.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/path.c b/path.c
index 03d284ba8..12ef731ac 100644
--- a/path.c
+++ b/path.c
@@ -222,7 +222,6 @@ int git_mkstemps_mode(char *pattern, int suffix_len, int mode)
}
/* We return the null string if we can't find a unique file name. */
pattern[0] = '\0';
- errno = EINVAL;
return -1;
}