diff options
author | Brandon Casey <casey@nrlssc.navy.mil> | 2008-02-08 20:32:47 -0600 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2008-02-11 18:25:10 -0800 |
commit | cba22528fa897728ebbffb95c05037ec9a20ea7c (patch) | |
tree | 2f1631cd5de7827b86847d98d9850e58d1d300c7 /compat/fopen.c | |
parent | 40aab8119f38c622f58d8e612e7a632eb1f3ded2 (diff) | |
download | git-cba22528fa897728ebbffb95c05037ec9a20ea7c.tar.gz git-cba22528fa897728ebbffb95c05037ec9a20ea7c.tar.xz |
Add compat/fopen.c which returns NULL on attempt to open directory
Some systems do not fail as expected when fread et al. are called on
a directory stream. Replace fopen on such systems which will fail
when the supplied path is a directory.
Signed-off-by: Brandon Casey <casey@nrlssc.navy.mil>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'compat/fopen.c')
-rw-r--r-- | compat/fopen.c | 26 |
1 files changed, 26 insertions, 0 deletions
diff --git a/compat/fopen.c b/compat/fopen.c new file mode 100644 index 000000000..ccb9e89fa --- /dev/null +++ b/compat/fopen.c @@ -0,0 +1,26 @@ +#include "../git-compat-util.h" +#undef fopen +FILE *git_fopen(const char *path, const char *mode) +{ + FILE *fp; + struct stat st; + + if (mode[0] == 'w' || mode[0] == 'a') + return fopen(path, mode); + + if (!(fp = fopen(path, mode))) + return NULL; + + if (fstat(fileno(fp), &st)) { + fclose(fp); + return NULL; + } + + if (S_ISDIR(st.st_mode)) { + fclose(fp); + errno = EISDIR; + return NULL; + } + + return fp; +} |