diff options
author | Paul Smith <psmith@gnu.org> | 2013-04-28 19:25:24 -0400 |
---|---|---|
committer | Paul Smith <psmith@gnu.org> | 2013-04-28 19:25:24 -0400 |
commit | 9253a34a57be0c6688278d120c450d1e81971a31 (patch) | |
tree | 83049a188c98569157a06cd8d6de26f55503bd0f /dir.c | |
parent | f88eb23b02088e7549c46231d0dfa4ee8d35a365 (diff) | |
parent | 5667376edc9cc68c1ed40f985c6d2174dfaeb64c (diff) | |
download | gunmake-9253a34a57be0c6688278d120c450d1e81971a31.tar.gz |
Merge branch 'master' of git.savannah.gnu.org:/srv/git/make
Diffstat (limited to 'dir.c')
-rw-r--r-- | dir.c | 22 |
1 files changed, 21 insertions, 1 deletions
@@ -1185,8 +1185,11 @@ ansi_free (void *p) /* On 64 bit ReliantUNIX (5.44 and above) in LFS mode, stat() is actually a * macro for stat64(). If stat is a macro, make a local wrapper function to * invoke it. + * + * On MS-Windows, stat() "succeeds" for foo/bar/. where foo/bar is a + * regular file; fix that here. */ -#ifndef stat +#if !defined(stat) && !defined(WINDOWS32) # ifndef VMS int stat (const char *path, struct stat *sbuf); # endif @@ -1196,6 +1199,23 @@ static int local_stat (const char *path, struct stat *buf) { int e; +#ifdef WINDOWS32 + size_t plen = strlen (path); + + /* Make sure the parent of "." exists and is a directory, not a + file. This is because 'stat' on Windows normalizes the argument + foo/. => foo without checking first that foo is a directory. */ + if (plen > 1 && path[plen - 1] == '.' + && (path[plen - 2] == '/' || path[plen - 2] == '\\')) + { + char parent[MAXPATHLEN]; + + strncpy (parent, path, plen - 2); + parent[plen - 2] = '\0'; + if (stat (parent, buf) < 0 || !_S_ISDIR (buf->st_mode)) + return -1; + } +#endif EINTRLOOP (e, stat (path, buf)); return e; |