diff options
author | Eli Zaretskii <eliz@gnu.org> | 2013-06-22 16:16:56 +0300 |
---|---|---|
committer | Eli Zaretskii <eliz@gnu.org> | 2013-06-22 16:16:56 +0300 |
commit | 5b65c5b916e458e5eed3c74f038dd11c008fdfd8 (patch) | |
tree | 9b6704597ca3d61b04cae5e36d4f0e9c0182ba13 | |
parent | cc85b927cdc1a4dad3217842215903a45044fc43 (diff) | |
download | gunmake-5b65c5b916e458e5eed3c74f038dd11c008fdfd8.tar.gz |
Fix a fatal error at startup on Windows due to non-ASCII characters in PATH.
main.c (find_and_set_default_shell): Don't use file_exists_p or
dir_file_exists_p, as those call readdir, which can fail if PATH
includes directories with non-ASCII characters, and that would
cause Make to fail at startup with confusing diagnostics. See
https://sourceforge.net/mailarchive/message.php?msg_id=30846737
for the details.
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | main.c | 21 |
2 files changed, 21 insertions, 9 deletions
@@ -1,3 +1,12 @@ +2013-06-22 Eli Zaretskii <eliz@gnu.org> + + * main.c (find_and_set_default_shell): Don't use file_exists_p or + dir_file_exists_p, as those call readdir, which can fail if PATH + includes directories with non-ASCII characters, and that would + cause Make to fail at startup with confusing diagnostics. See + https://sourceforge.net/mailarchive/message.php?msg_id=30846737 + for the details. + 2013-06-22 Paul Smith <psmith@gnu.org> Improve performance by using a character map to determine where we @@ -956,7 +956,7 @@ find_and_set_default_shell (const char *token) /* no new information, path already set or known */ sh_found = 1; } - else if (file_exists_p (search_token)) + else if (_access (search_token, 0) == 0) { /* search token path was found */ sprintf (sh_path, "%s", search_token); @@ -982,9 +982,9 @@ find_and_set_default_shell (const char *token) { *ep = '\0'; - if (dir_file_exists_p (p, search_token)) + sprintf (sh_path, "%s/%s", p, search_token); + if (_access (sh_path, 0) == 0) { - sprintf (sh_path, "%s/%s", p, search_token); default_shell = xstrdup (w32ify (sh_path, 0)); sh_found = 1; *ep = PATH_SEPARATOR_CHAR; @@ -1002,12 +1002,15 @@ find_and_set_default_shell (const char *token) } /* be sure to check last element of Path */ - if (p && *p && dir_file_exists_p (p, search_token)) - { - sprintf (sh_path, "%s/%s", p, search_token); - default_shell = xstrdup (w32ify (sh_path, 0)); - sh_found = 1; - } + if (p && *p) + { + sprintf (sh_path, "%s/%s", p, search_token); + if (_access (sh_path, 0) == 0) + { + default_shell = xstrdup (w32ify (sh_path, 0)); + sh_found = 1; + } + } if (sh_found) DB (DB_VERBOSE, |