diff options
author | Paul Eggert <eggert@cs.ucla.edu> | 2005-01-20 22:17:26 +0000 |
---|---|---|
committer | Paul Eggert <eggert@cs.ucla.edu> | 2005-01-20 22:17:26 +0000 |
commit | 7e43a4e6c4994afac9ae1591d67e1436eeed602b (patch) | |
tree | da8181febd36f5e8ff9251d7a10cb657eaba831a /lib | |
parent | 2b631e90a8027bd90e55d685ad933113f4b7e7ba (diff) | |
download | coreutils-7e43a4e6c4994afac9ae1591d67e1436eeed602b.tar.xz |
(save_cwd): Remove code to support the case
where fchdir is missing or flaky.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/save-cwd.c | 67 |
1 files changed, 17 insertions, 50 deletions
diff --git a/lib/save-cwd.c b/lib/save-cwd.c index 272f7421e..596b78c8d 100644 --- a/lib/save-cwd.c +++ b/lib/save-cwd.c @@ -1,5 +1,7 @@ /* save-cwd.c -- Save and restore current working directory. - Copyright (C) 1995, 1997, 1998, 2003, 2004 Free Software Foundation, Inc. + + Copyright (C) 1995, 1997, 1998, 2003, 2004, 2005 Free Software + Foundation, Inc. This program is free software; you can redistribute it and/or modify it under the terms of the GNU General Public License as published by @@ -51,68 +53,33 @@ closed; return non-zero -- in that case, free_cwd need not be called, but doing so is ok. Otherwise, return zero. - The `raison d'etre' for this interface is that some systems lack - support for fchdir, and getcwd is not robust or as efficient. + The `raison d'etre' for this interface is that the working directory + is sometimes inaccessible, and getcwd is not robust or as efficient. So, we prefer to use the open/fchdir approach, but fall back on - getcwd if necessary. Some systems lack fchdir altogether: OS/2, - Cygwin (as of March 2003), SCO Xenix. At least SunOS 4 and Irix 5.3 - provide the function, yet it doesn't work for partitions on which - auditing is enabled. */ + getcwd if necessary. + + Some systems lack fchdir altogether: e.g., OS/2, pre-2001 Cygwin, + SCO Xenix. Also, SunOS 4 and Irix 5.3 provide the function, yet it + doesn't work for partitions on which auditing is enabled. If + you're still using an obsolete system with these problems, please + send email to the maintainer of this code. */ int save_cwd (struct saved_cwd *cwd) { -#if !HAVE_FCHDIR -# undef fchdir -# define fchdir(x) (abort (), 0) - bool have_working_fchdir = false; - bool fchdir_needs_testing = false; -#elif (__sgi || __sun) - static bool have_working_fchdir = true; - bool fchdir_needs_testing = true; -#else - bool have_working_fchdir = true; - bool fchdir_needs_testing = false; -#endif - - cwd->desc = -1; cwd->name = NULL; - if (have_working_fchdir) + cwd->desc = open (".", O_RDONLY); + if (cwd->desc < 0) { - cwd->desc = open (".", O_RDONLY); + cwd->desc = open (".", O_WRONLY); if (cwd->desc < 0) { - cwd->desc = open (".", O_WRONLY); - if (cwd->desc < 0) - { - cwd->name = xgetcwd (); - return cwd->name ? 0 : -1; - } - } - - /* On SunOS 4 and IRIX 5.3, fchdir returns EINVAL when auditing - is enabled, so we have to fall back to chdir. */ - if (fchdir_needs_testing && fchdir (cwd->desc) != 0) - { - int saved_errno = errno; - close (cwd->desc); - cwd->desc = -1; - if (saved_errno != EINVAL) - { - errno = saved_errno; - return -1; - } - have_working_fchdir = false; + cwd->name = xgetcwd (); + return cwd->name ? 0 : -1; } } - if (!have_working_fchdir) - { - cwd->name = xgetcwd (); - if (cwd->name == NULL) - return -1; - } return 0; } |