Mercurial > gnulib
annotate lib/fts.c @ 11924:10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
* lib/fts.c (fts_close): Detect close failure, not just fchdir failure,
and propagate to caller, along with errno.
| author | Jim Meyering <meyering@redhat.com> |
|---|---|
| date | Tue, 01 Sep 2009 11:18:07 +0200 |
| parents | 92457e8bc574 |
| children | 7cbcde229d97 |
| rev | line source |
|---|---|
| 5858 | 1 /* Traverse a file hierarchy. |
| 2 | |
|
11163
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
3 Copyright (C) 2004-2009 Free Software Foundation, Inc. |
| 5858 | 4 |
|
9309
bbbbbf4cd1c5
Change copyright notice from GPLv2+ to GPLv3+.
Bruno Haible <bruno@clisp.org>
parents:
9129
diff
changeset
|
5 This program is free software: you can redistribute it and/or modify |
| 5858 | 6 it under the terms of the GNU General Public License as published by |
|
9309
bbbbbf4cd1c5
Change copyright notice from GPLv2+ to GPLv3+.
Bruno Haible <bruno@clisp.org>
parents:
9129
diff
changeset
|
7 the Free Software Foundation; either version 3 of the License, or |
|
bbbbbf4cd1c5
Change copyright notice from GPLv2+ to GPLv3+.
Bruno Haible <bruno@clisp.org>
parents:
9129
diff
changeset
|
8 (at your option) any later version. |
| 5858 | 9 |
| 10 This program is distributed in the hope that it will be useful, | |
| 11 but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 12 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 13 GNU General Public License for more details. | |
| 14 | |
| 15 You should have received a copy of the GNU General Public License | |
|
9309
bbbbbf4cd1c5
Change copyright notice from GPLv2+ to GPLv3+.
Bruno Haible <bruno@clisp.org>
parents:
9129
diff
changeset
|
16 along with this program. If not, see <http://www.gnu.org/licenses/>. */ |
| 5858 | 17 |
| 18 /*- | |
| 19 * Copyright (c) 1990, 1993, 1994 | |
| 20 * The Regents of the University of California. All rights reserved. | |
| 21 * | |
| 22 * Redistribution and use in source and binary forms, with or without | |
| 23 * modification, are permitted provided that the following conditions | |
| 24 * are met: | |
| 25 * 1. Redistributions of source code must retain the above copyright | |
| 26 * notice, this list of conditions and the following disclaimer. | |
| 27 * 2. Redistributions in binary form must reproduce the above copyright | |
| 28 * notice, this list of conditions and the following disclaimer in the | |
| 29 * documentation and/or other materials provided with the distribution. | |
| 30 * 4. Neither the name of the University nor the names of its contributors | |
| 31 * may be used to endorse or promote products derived from this software | |
| 32 * without specific prior written permission. | |
| 33 * | |
| 34 * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND | |
| 35 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE | |
| 36 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE | |
| 37 * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE | |
| 38 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL | |
| 39 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS | |
| 40 * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) | |
| 41 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT | |
| 42 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY | |
| 43 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF | |
| 44 * SUCH DAMAGE. | |
| 45 */ | |
| 46 | |
|
7302
8a1a9361108c
* _fpending.c: Include <config.h> unconditionally, since we no
Paul Eggert <eggert@cs.ucla.edu>
parents:
7225
diff
changeset
|
47 #include <config.h> |
| 5858 | 48 |
| 49 #if defined(LIBC_SCCS) && !defined(lint) | |
| 50 static char sccsid[] = "@(#)fts.c 8.6 (Berkeley) 8/14/94"; | |
| 51 #endif /* LIBC_SCCS and not lint */ | |
| 52 | |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
53 #include "fts_.h" |
|
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
54 |
| 5858 | 55 #if HAVE_SYS_PARAM_H || defined _LIBC |
| 56 # include <sys/param.h> | |
| 57 #endif | |
| 58 #ifdef _LIBC | |
| 59 # include <include/sys/stat.h> | |
| 60 #else | |
| 61 # include <sys/stat.h> | |
| 62 #endif | |
| 63 #include <fcntl.h> | |
| 64 #include <errno.h> | |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
65 #include <stdbool.h> |
| 5858 | 66 #include <stdlib.h> |
| 67 #include <string.h> | |
| 68 #include <unistd.h> | |
| 69 | |
|
6034
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
70 #if ! _LIBC |
| 7151 | 71 # include "fcntl--.h" |
| 72 # include "openat.h" | |
| 73 # include "unistd--.h" | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
74 # include "same-inode.h" |
|
6034
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
75 #endif |
|
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
76 |
|
6949
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
77 #include <dirent.h> |
|
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
78 #ifndef _D_EXACT_NAMLEN |
|
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
79 # define _D_EXACT_NAMLEN(dirent) strlen ((dirent)->d_name) |
| 5858 | 80 #endif |
| 81 | |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
82 #if HAVE_STRUCT_DIRENT_D_TYPE |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
83 /* True if the type of the directory entry D is known. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
84 # define DT_IS_KNOWN(d) ((d)->d_type != DT_UNKNOWN) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
85 /* True if the type of the directory entry D must be T. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
86 # define DT_MUST_BE(d, t) ((d)->d_type == (t)) |
|
10843
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
87 # define D_TYPE(d) ((d)->d_type) |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
88 #else |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
89 # define DT_IS_KNOWN(d) false |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
90 # define DT_MUST_BE(d, t) false |
|
10843
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
91 # define D_TYPE(d) DT_UNKNOWN |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
92 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
93 # undef DT_UNKNOWN |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
94 # define DT_UNKNOWN 0 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
95 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
96 /* Any nonzero values will do here, so long as they're distinct. |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
97 Undef any existing macros out of the way. */ |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
98 # undef DT_BLK |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
99 # undef DT_CHR |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
100 # undef DT_DIR |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
101 # undef DT_FIFO |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
102 # undef DT_LNK |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
103 # undef DT_REG |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
104 # undef DT_SOCK |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
105 # define DT_BLK 1 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
106 # define DT_CHR 2 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
107 # define DT_DIR 3 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
108 # define DT_FIFO 4 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
109 # define DT_LNK 5 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
110 # define DT_REG 6 |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
111 # define DT_SOCK 7 |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
112 #endif |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
113 |
|
10863
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
114 #ifndef S_IFLNK |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
115 # define S_IFLNK 0 |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
116 #endif |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
117 #ifndef S_IFSOCK |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
118 # define S_IFSOCK 0 |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
119 #endif |
|
054d026819f0
work around mingw's lack of some S_IF definitions
Jim Meyering <meyering@redhat.com>
parents:
10844
diff
changeset
|
120 |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
121 enum |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
122 { |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
123 NOT_AN_INODE_NUMBER = 0 |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
124 }; |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
125 |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
126 #ifdef D_INO_IN_DIRENT |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
127 # define D_INO(dp) (dp)->d_ino |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
128 #else |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
129 /* Some systems don't have inodes, so fake them to avoid lots of ifdefs. */ |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
130 # define D_INO(dp) NOT_AN_INODE_NUMBER |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
131 #endif |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
132 |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
133 /* If there are more than this many entries in a directory, |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
134 and the conditions mentioned below are satisfied, then sort |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
135 the entries on inode number before any further processing. */ |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
136 #ifndef FTS_INODE_SORT_DIR_ENTRIES_THRESHOLD |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
137 # define FTS_INODE_SORT_DIR_ENTRIES_THRESHOLD 10000 |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
138 #endif |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
139 enum |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
140 { |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
141 _FTS_INODE_SORT_DIR_ENTRIES_THRESHOLD = FTS_INODE_SORT_DIR_ENTRIES_THRESHOLD |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
142 }; |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
143 |
|
7684
784e80c3d895
Work around a compile-time error from the HP-UX 11.00 /bin/cc.
Jim Meyering <jim@meyering.net>
parents:
7681
diff
changeset
|
144 enum Fts_stat |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
145 { |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
146 FTS_NO_STAT_REQUIRED = 1, |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
147 FTS_STAT_REQUIRED = 2 |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
148 }; |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
149 |
| 5858 | 150 #ifdef _LIBC |
| 151 # undef close | |
| 152 # define close __close | |
| 153 # undef closedir | |
| 154 # define closedir __closedir | |
| 155 # undef fchdir | |
| 156 # define fchdir __fchdir | |
| 157 # undef open | |
| 158 # define open __open | |
| 159 # undef opendir | |
| 160 # define opendir __opendir | |
| 161 # undef readdir | |
| 162 # define readdir __readdir | |
| 163 #else | |
| 164 # undef internal_function | |
| 165 # define internal_function /* empty */ | |
| 166 #endif | |
| 167 | |
| 168 #ifndef __set_errno | |
| 169 # define __set_errno(Val) errno = (Val) | |
| 170 #endif | |
| 171 | |
| 172 #ifndef __attribute__ | |
|
9613
c4fa39bf5223
Don't redefine __attribute__ without a need.
Bruno Haible <bruno@clisp.org>
parents:
9562
diff
changeset
|
173 # if __GNUC__ < 2 || (__GNUC__ == 2 && __GNUC_MINOR__ < 8) |
| 5858 | 174 # define __attribute__(x) /* empty */ |
| 175 # endif | |
| 176 #endif | |
| 177 | |
| 178 #ifndef ATTRIBUTE_UNUSED | |
| 179 # define ATTRIBUTE_UNUSED __attribute__ ((__unused__)) | |
| 180 #endif | |
| 181 | |
| 7151 | 182 /* If this host provides the openat function, then we can avoid |
| 183 attempting to open "." in some initialization code below. */ | |
| 184 #ifdef HAVE_OPENAT | |
| 185 # define HAVE_OPENAT_SUPPORT 1 | |
| 186 #else | |
| 187 # define HAVE_OPENAT_SUPPORT 0 | |
| 188 #endif | |
| 5858 | 189 |
|
7785
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
190 #ifdef NDEBUG |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
191 # define fts_assert(expr) ((void) 0) |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
192 #else |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
193 # define fts_assert(expr) \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
194 do \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
195 { \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
196 if (!(expr)) \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
197 abort (); \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
198 } \ |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
199 while (false) |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
200 #endif |
|
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
201 |
| 5858 | 202 static FTSENT *fts_alloc (FTS *, const char *, size_t) internal_function; |
| 203 static FTSENT *fts_build (FTS *, int) internal_function; | |
| 204 static void fts_lfree (FTSENT *) internal_function; | |
| 205 static void fts_load (FTS *, FTSENT *) internal_function; | |
| 206 static size_t fts_maxarglen (char * const *) internal_function; | |
| 207 static void fts_padjust (FTS *, FTSENT *) internal_function; | |
| 208 static bool fts_palloc (FTS *, size_t) internal_function; | |
| 209 static FTSENT *fts_sort (FTS *, FTSENT *, size_t) internal_function; | |
| 210 static unsigned short int fts_stat (FTS *, FTSENT *, bool) internal_function; | |
| 211 static int fts_safe_changedir (FTS *, FTSENT *, int, const char *) | |
| 212 internal_function; | |
| 213 | |
|
7797
10432bdf90d3
2007-01-08 Bruno Haible <bruno@clisp.org>
Bruno Haible <bruno@clisp.org>
parents:
7786
diff
changeset
|
214 #if GNULIB_FTS |
|
10432bdf90d3
2007-01-08 Bruno Haible <bruno@clisp.org>
Bruno Haible <bruno@clisp.org>
parents:
7786
diff
changeset
|
215 # include "fts-cycle.c" |
|
10432bdf90d3
2007-01-08 Bruno Haible <bruno@clisp.org>
Bruno Haible <bruno@clisp.org>
parents:
7786
diff
changeset
|
216 #else |
| 5872 | 217 static bool enter_dir (FTS *fts, FTSENT *ent) { return true; } |
| 218 static void leave_dir (FTS *fts, FTSENT *ent) {} | |
| 219 static bool setup_dir (FTS *fts) { return true; } | |
| 220 static void free_dir (FTS *fts) {} | |
| 221 #endif | |
| 222 | |
| 5858 | 223 #ifndef MAX |
| 224 # define MAX(a,b) ((a) > (b) ? (a) : (b)) | |
| 225 #endif | |
| 226 | |
| 227 #ifndef SIZE_MAX | |
| 228 # define SIZE_MAX ((size_t) -1) | |
| 229 #endif | |
| 230 | |
| 231 #ifndef O_DIRECTORY | |
| 232 # define O_DIRECTORY 0 | |
| 233 #endif | |
| 234 | |
| 235 #define ISDOT(a) (a[0] == '.' && (!a[1] || (a[1] == '.' && !a[2]))) | |
| 7151 | 236 #define STREQ(a, b) (strcmp ((a), (b)) == 0) |
| 5858 | 237 |
| 238 #define CLR(opt) (sp->fts_options &= ~(opt)) | |
| 239 #define ISSET(opt) (sp->fts_options & (opt)) | |
| 240 #define SET(opt) (sp->fts_options |= (opt)) | |
| 241 | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
242 /* FIXME: make this a function */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
243 #define RESTORE_INITIAL_CWD(sp) \ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
244 (fd_ring_clear (&((sp)->fts_fd_ring)), \ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
245 FCHDIR ((sp), (ISSET (FTS_CWDFD) ? AT_FDCWD : (sp)->fts_rfd))) |
| 7151 | 246 |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
247 /* FIXME: FTS_NOCHDIR is now misnamed. |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
248 Call it FTS_USE_FULL_RELATIVE_FILE_NAMES instead. */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
249 #define FCHDIR(sp, fd) \ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
250 (!ISSET(FTS_NOCHDIR) && (ISSET(FTS_CWDFD) \ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
251 ? (cwd_advance_fd ((sp), (fd), true), 0) \ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
252 : fchdir (fd))) |
| 7151 | 253 |
| 5858 | 254 |
| 255 /* fts_build flags */ | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
256 /* FIXME: make this an enum */ |
| 5858 | 257 #define BCHILD 1 /* fts_children */ |
| 258 #define BNAMES 2 /* fts_children, names only */ | |
| 259 #define BREAD 3 /* fts_read */ | |
| 260 | |
| 261 #if FTS_DEBUG | |
|
5894
d09af5fbd9f2
* modules/fts (Files): Remove m4/inttypes-pri.m4.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5877
diff
changeset
|
262 # include <inttypes.h> |
|
d09af5fbd9f2
* modules/fts (Files): Remove m4/inttypes-pri.m4.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5877
diff
changeset
|
263 # include <stdint.h> |
|
d09af5fbd9f2
* modules/fts (Files): Remove m4/inttypes-pri.m4.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5877
diff
changeset
|
264 # include <stdio.h> |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
265 # include "getcwdat.h" |
| 5858 | 266 bool fts_debug = false; |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
267 # define Dprintf(x) do { if (fts_debug) printf x; } while (false) |
| 5858 | 268 #else |
| 269 # define Dprintf(x) | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
270 # define fd_ring_check(x) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
271 # define fd_ring_print(a, b, c) |
| 5858 | 272 #endif |
| 273 | |
| 274 #define LEAVE_DIR(Fts, Ent, Tag) \ | |
| 5872 | 275 do \ |
| 276 { \ | |
| 5858 | 277 Dprintf ((" %s-leaving: %s\n", Tag, (Ent)->fts_path)); \ |
| 5872 | 278 leave_dir (Fts, Ent); \ |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
279 fd_ring_check (Fts); \ |
| 5872 | 280 } \ |
| 281 while (false) | |
| 5858 | 282 |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
283 static void |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
284 fd_ring_clear (I_ring *fd_ring) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
285 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
286 while ( ! i_ring_empty (fd_ring)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
287 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
288 int fd = i_ring_pop (fd_ring); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
289 if (0 <= fd) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
290 close (fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
291 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
292 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
293 |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
294 /* Overload the fts_statp->st_size member (otherwise unused, when |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
295 fts_info is FTS_NSOK) to indicate whether fts_read should stat |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
296 this entry or not. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
297 static void |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
298 fts_set_stat_required (FTSENT *p, bool required) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
299 { |
|
7785
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
300 fts_assert (p->fts_info == FTS_NSOK); |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
301 p->fts_statp->st_size = (required |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
302 ? FTS_STAT_REQUIRED |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
303 : FTS_NO_STAT_REQUIRED); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
304 } |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
305 |
| 7151 | 306 /* file-descriptor-relative opendir. */ |
| 307 /* FIXME: if others need this function, move it into lib/openat.c */ | |
| 308 static inline DIR * | |
| 309 internal_function | |
| 310 opendirat (int fd, char const *dir) | |
| 311 { | |
| 312 int new_fd = openat (fd, dir, O_RDONLY); | |
| 313 DIR *dirp; | |
| 314 | |
| 315 if (new_fd < 0) | |
| 316 return NULL; | |
| 317 dirp = fdopendir (new_fd); | |
| 318 if (dirp == NULL) | |
| 319 { | |
| 320 int saved_errno = errno; | |
| 321 close (new_fd); | |
| 322 errno = saved_errno; | |
| 323 } | |
| 324 return dirp; | |
| 325 } | |
| 326 | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
327 /* Virtual fchdir. Advance SP's working directory file descriptor, |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
328 SP->fts_cwd_fd, to FD, and push the previous value onto the fd_ring. |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
329 CHDIR_DOWN_ONE is true if FD corresponds to an entry in the directory |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
330 open on sp->fts_cwd_fd; i.e., to move the working directory one level |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
331 down. */ |
| 7151 | 332 static void |
| 333 internal_function | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
334 cwd_advance_fd (FTS *sp, int fd, bool chdir_down_one) |
| 7151 | 335 { |
| 336 int old = sp->fts_cwd_fd; | |
|
7785
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
337 fts_assert (old != fd || old == AT_FDCWD); |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
338 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
339 if (chdir_down_one) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
340 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
341 /* Push "old" onto the ring. |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
342 If the displaced file descriptor is non-negative, close it. */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
343 int prev_fd_in_slot = i_ring_push (&sp->fts_fd_ring, old); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
344 fd_ring_print (sp, stderr, "post-push"); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
345 if (0 <= prev_fd_in_slot) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
346 close (prev_fd_in_slot); /* ignore any close failure */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
347 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
348 else if ( ! ISSET (FTS_NOCHDIR)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
349 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
350 if (0 <= old) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
351 close (old); /* ignore any close failure */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
352 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
353 |
| 7151 | 354 sp->fts_cwd_fd = fd; |
| 355 } | |
| 356 | |
| 5858 | 357 /* Open the directory DIR if possible, and return a file |
| 358 descriptor. Return -1 and set errno on failure. It doesn't matter | |
| 359 whether the file descriptor has read or write access. */ | |
| 360 | |
| 7151 | 361 static inline int |
| 5858 | 362 internal_function |
| 7151 | 363 diropen (FTS const *sp, char const *dir) |
| 5858 | 364 { |
| 7151 | 365 int open_flags = (O_RDONLY | O_DIRECTORY | O_NOCTTY | O_NONBLOCK |
| 366 | (ISSET (FTS_PHYSICAL) ? O_NOFOLLOW : 0)); | |
| 367 | |
| 368 return (ISSET (FTS_CWDFD) | |
| 369 ? openat (sp->fts_cwd_fd, dir, open_flags) | |
| 370 : open (dir, open_flags)); | |
| 5858 | 371 } |
| 372 | |
| 373 FTS * | |
| 374 fts_open (char * const *argv, | |
| 375 register int options, | |
| 376 int (*compar) (FTSENT const **, FTSENT const **)) | |
| 377 { | |
| 378 register FTS *sp; | |
| 379 register FTSENT *p, *root; | |
| 380 register size_t nitems; | |
| 7151 | 381 FTSENT *parent = NULL; |
| 382 FTSENT *tmp = NULL; /* pacify gcc */ | |
| 5858 | 383 size_t len; |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
384 bool defer_stat; |
| 5858 | 385 |
| 386 /* Options check. */ | |
| 387 if (options & ~FTS_OPTIONMASK) { | |
| 388 __set_errno (EINVAL); | |
| 389 return (NULL); | |
| 390 } | |
| 7151 | 391 if ((options & FTS_NOCHDIR) && (options & FTS_CWDFD)) { |
| 392 __set_errno (EINVAL); | |
| 393 return (NULL); | |
| 394 } | |
| 395 if ( ! (options & (FTS_LOGICAL | FTS_PHYSICAL))) { | |
| 396 __set_errno (EINVAL); | |
| 397 return (NULL); | |
| 398 } | |
| 5858 | 399 |
| 400 /* Allocate/initialize the stream */ | |
| 401 if ((sp = malloc(sizeof(FTS))) == NULL) | |
| 402 return (NULL); | |
| 403 memset(sp, 0, sizeof(FTS)); | |
| 404 sp->fts_compar = compar; | |
| 405 sp->fts_options = options; | |
| 406 | |
| 407 /* Logical walks turn on NOCHDIR; symbolic links are too hard. */ | |
| 7151 | 408 if (ISSET(FTS_LOGICAL)) { |
| 5858 | 409 SET(FTS_NOCHDIR); |
| 7151 | 410 CLR(FTS_CWDFD); |
| 411 } | |
| 412 | |
| 413 /* Initialize fts_cwd_fd. */ | |
| 414 sp->fts_cwd_fd = AT_FDCWD; | |
| 415 if ( ISSET(FTS_CWDFD) && ! HAVE_OPENAT_SUPPORT) | |
| 416 { | |
| 417 /* While it isn't technically necessary to open "." this | |
| 418 early, doing it here saves us the trouble of ensuring | |
| 419 later (where it'd be messier) that "." can in fact | |
| 420 be opened. If not, revert to FTS_NOCHDIR mode. */ | |
| 421 int fd = open (".", O_RDONLY); | |
| 422 if (fd < 0) | |
| 423 { | |
| 424 /* Even if `.' is unreadable, don't revert to FTS_NOCHDIR mode | |
| 425 on systems like Linux+PROC_FS, where our openat emulation | |
| 426 is good enough. Note: on a system that emulates | |
| 427 openat via /proc, this technique can still fail, but | |
| 428 only in extreme conditions, e.g., when the working | |
| 429 directory cannot be saved (i.e. save_cwd fails) -- | |
|
7402
82dc76cfd521
* fts.c (fts_open): Tiny comment change.
Jim Meyering <jim@meyering.net>
parents:
7396
diff
changeset
|
430 and that happens on Linux only when "." is unreadable |
| 7151 | 431 and the CWD would be longer than PATH_MAX. |
| 432 FIXME: once Linux kernel openat support is well established, | |
| 433 replace the above open call and this entire if/else block | |
| 434 with the body of the if-block below. */ | |
| 435 if ( openat_needs_fchdir ()) | |
| 436 { | |
| 437 SET(FTS_NOCHDIR); | |
| 438 CLR(FTS_CWDFD); | |
| 439 } | |
| 440 } | |
| 441 else | |
| 442 { | |
| 443 close (fd); | |
| 444 } | |
| 445 } | |
| 5858 | 446 |
| 447 /* | |
| 5907 | 448 * Start out with 1K of file name space, and enough, in any case, |
| 449 * to hold the user's file names. | |
| 5858 | 450 */ |
| 451 #ifndef MAXPATHLEN | |
| 452 # define MAXPATHLEN 1024 | |
| 453 #endif | |
|
6611
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
454 { |
|
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
455 size_t maxarglen = fts_maxarglen(argv); |
|
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
456 if (! fts_palloc(sp, MAX(maxarglen, MAXPATHLEN))) |
|
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
457 goto mem1; |
|
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
458 } |
| 5858 | 459 |
| 460 /* Allocate/initialize root's parent. */ | |
| 7151 | 461 if (*argv != NULL) { |
| 462 if ((parent = fts_alloc(sp, "", 0)) == NULL) | |
| 463 goto mem2; | |
| 464 parent->fts_level = FTS_ROOTPARENTLEVEL; | |
| 465 } | |
| 5858 | 466 |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
467 /* The classic fts implementation would call fts_stat with |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
468 a new entry for each iteration of the loop below. |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
469 If the comparison function is not specified or if the |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
470 FTS_DEFER_STAT option is in effect, don't stat any entry |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
471 in this loop. This is an attempt to minimize the interval |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
472 between the initial stat/lstat/fstatat and the point at which |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
473 a directory argument is first opened. This matters for any |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
474 directory command line argument that resides on a file system |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
475 without genuine i-nodes. If you specify FTS_DEFER_STAT along |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
476 with a comparison function, that function must not access any |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
477 data via the fts_statp pointer. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
478 defer_stat = (compar == NULL || ISSET(FTS_DEFER_STAT)); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
479 |
| 5858 | 480 /* Allocate/initialize root(s). */ |
| 481 for (root = NULL, nitems = 0; *argv != NULL; ++argv, ++nitems) { | |
| 5907 | 482 /* Don't allow zero-length file names. */ |
| 5858 | 483 if ((len = strlen(*argv)) == 0) { |
| 484 __set_errno (ENOENT); | |
| 485 goto mem3; | |
| 486 } | |
| 487 | |
| 488 if ((p = fts_alloc(sp, *argv, len)) == NULL) | |
| 489 goto mem3; | |
| 490 p->fts_level = FTS_ROOTLEVEL; | |
| 491 p->fts_parent = parent; | |
| 492 p->fts_accpath = p->fts_name; | |
|
7499
67ce0bed0a20
Fix a bug in yesterday's change.
Jim Meyering <jim@meyering.net>
parents:
7483
diff
changeset
|
493 /* Even when defer_stat is true, be sure to stat the first |
|
67ce0bed0a20
Fix a bug in yesterday's change.
Jim Meyering <jim@meyering.net>
parents:
7483
diff
changeset
|
494 command line argument, since fts_read (at least with |
|
67ce0bed0a20
Fix a bug in yesterday's change.
Jim Meyering <jim@meyering.net>
parents:
7483
diff
changeset
|
495 FTS_XDEV) requires that. */ |
|
67ce0bed0a20
Fix a bug in yesterday's change.
Jim Meyering <jim@meyering.net>
parents:
7483
diff
changeset
|
496 if (defer_stat && root != NULL) { |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
497 p->fts_info = FTS_NSOK; |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
498 fts_set_stat_required(p, true); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
499 } else { |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
500 p->fts_info = fts_stat(sp, p, false); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
501 } |
| 5858 | 502 |
| 503 /* | |
| 504 * If comparison routine supplied, traverse in sorted | |
| 505 * order; otherwise traverse in the order specified. | |
| 506 */ | |
| 507 if (compar) { | |
| 508 p->fts_link = root; | |
| 509 root = p; | |
| 510 } else { | |
| 511 p->fts_link = NULL; | |
| 512 if (root == NULL) | |
| 513 tmp = root = p; | |
| 514 else { | |
| 515 tmp->fts_link = p; | |
| 516 tmp = p; | |
| 517 } | |
| 518 } | |
| 519 } | |
| 520 if (compar && nitems > 1) | |
| 521 root = fts_sort(sp, root, nitems); | |
| 522 | |
| 523 /* | |
| 524 * Allocate a dummy pointer and make fts_read think that we've just | |
| 525 * finished the node before the root(s); set p->fts_info to FTS_INIT | |
| 526 * so that everything about the "current" node is ignored. | |
| 527 */ | |
| 528 if ((sp->fts_cur = fts_alloc(sp, "", 0)) == NULL) | |
| 529 goto mem3; | |
| 530 sp->fts_cur->fts_link = root; | |
| 531 sp->fts_cur->fts_info = FTS_INIT; | |
| 5872 | 532 if (! setup_dir (sp)) |
|
7483
2433e5f6bfd0
* lib/fts.c (fts_open): Use consistent indentation.
Jim Meyering <jim@meyering.net>
parents:
7482
diff
changeset
|
533 goto mem3; |
| 5858 | 534 |
| 535 /* | |
| 536 * If using chdir(2), grab a file descriptor pointing to dot to ensure | |
| 5907 | 537 * that we can get back here; this could be avoided for some file names, |
| 5858 | 538 * but almost certainly not worth the effort. Slashes, symbolic links, |
| 539 * and ".." are all fairly nasty problems. Note, if we can't get the | |
| 540 * descriptor we run anyway, just more slowly. | |
| 541 */ | |
| 7151 | 542 if (!ISSET(FTS_NOCHDIR) && !ISSET(FTS_CWDFD) |
| 543 && (sp->fts_rfd = diropen (sp, ".")) < 0) | |
| 5858 | 544 SET(FTS_NOCHDIR); |
| 545 | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
546 i_ring_init (&sp->fts_fd_ring, -1); |
| 5858 | 547 return (sp); |
| 548 | |
| 549 mem3: fts_lfree(root); | |
| 550 free(parent); | |
| 551 mem2: free(sp->fts_path); | |
| 552 mem1: free(sp); | |
| 553 return (NULL); | |
| 554 } | |
| 555 | |
| 556 static void | |
| 557 internal_function | |
| 558 fts_load (FTS *sp, register FTSENT *p) | |
| 559 { | |
| 560 register size_t len; | |
| 561 register char *cp; | |
| 562 | |
| 563 /* | |
| 564 * Load the stream structure for the next traversal. Since we don't | |
| 565 * actually enter the directory until after the preorder visit, set | |
| 566 * the fts_accpath field specially so the chdir gets done to the right | |
| 567 * place and the user can access the first node. From fts_open it's | |
| 5907 | 568 * known that the file name will fit. |
| 5858 | 569 */ |
| 570 len = p->fts_pathlen = p->fts_namelen; | |
| 571 memmove(sp->fts_path, p->fts_name, len + 1); | |
| 572 if ((cp = strrchr(p->fts_name, '/')) && (cp != p->fts_name || cp[1])) { | |
| 573 len = strlen(++cp); | |
| 574 memmove(p->fts_name, cp, len + 1); | |
| 575 p->fts_namelen = len; | |
| 576 } | |
| 577 p->fts_accpath = p->fts_path = sp->fts_path; | |
| 578 } | |
| 579 | |
| 580 int | |
| 581 fts_close (FTS *sp) | |
| 582 { | |
| 583 register FTSENT *freep, *p; | |
| 584 int saved_errno = 0; | |
| 585 | |
| 586 /* | |
| 587 * This still works if we haven't read anything -- the dummy structure | |
| 588 * points to the root list, so we step through to the end of the root | |
| 589 * list which has a valid parent pointer. | |
| 590 */ | |
| 591 if (sp->fts_cur) { | |
| 592 for (p = sp->fts_cur; p->fts_level >= FTS_ROOTLEVEL;) { | |
| 593 freep = p; | |
| 594 p = p->fts_link != NULL ? p->fts_link : p->fts_parent; | |
| 595 free(freep); | |
| 596 } | |
| 597 free(p); | |
| 598 } | |
| 599 | |
| 5907 | 600 /* Free up child linked list, sort array, file name buffer. */ |
| 5858 | 601 if (sp->fts_child) |
| 602 fts_lfree(sp->fts_child); | |
|
7396
6f79ccad009d
2006-10-02 Dmitry V. Levin <ldv@altlinux.org>
Jim Meyering <jim@meyering.net>
parents:
7302
diff
changeset
|
603 free(sp->fts_array); |
| 5858 | 604 free(sp->fts_path); |
| 605 | |
| 7151 | 606 if (ISSET(FTS_CWDFD)) |
| 607 { | |
| 608 if (0 <= sp->fts_cwd_fd) | |
|
11924
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
609 if (close (sp->fts_cwd_fd)) |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
610 saved_errno = errno; |
| 7151 | 611 } |
| 612 else if (!ISSET(FTS_NOCHDIR)) | |
| 613 { | |
| 614 /* Return to original directory, save errno if necessary. */ | |
| 615 if (fchdir(sp->fts_rfd)) | |
| 616 saved_errno = errno; | |
|
11924
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
617 |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
618 /* If close fails, record errno only if saved_errno is zero, |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
619 so that we report the probably-more-meaningful fchdir errno. */ |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
620 if (close (sp->fts_rfd)) |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
621 if (saved_errno == 0) |
|
10d064d93551
fts: fts_close now fails also when closing a dir file descriptor fails
Jim Meyering <meyering@redhat.com>
parents:
11722
diff
changeset
|
622 saved_errno = errno; |
| 7151 | 623 } |
| 5858 | 624 |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
625 fd_ring_clear (&sp->fts_fd_ring); |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
626 |
|
11169
cc4b6d391fce
fts: add #if guards so that the fts_lgpl module still builds
Jim Meyering <meyering@redhat.com>
parents:
11164
diff
changeset
|
627 #if GNULIB_FTS |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
628 if (sp->fts_leaf_optimization_works_ht) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
629 hash_free (sp->fts_leaf_optimization_works_ht); |
|
11169
cc4b6d391fce
fts: add #if guards so that the fts_lgpl module still builds
Jim Meyering <meyering@redhat.com>
parents:
11164
diff
changeset
|
630 #endif |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
631 |
| 5872 | 632 free_dir (sp); |
| 5858 | 633 |
| 634 /* Free up the stream pointer. */ | |
| 635 free(sp); | |
| 636 | |
| 637 /* Set errno and return. */ | |
| 638 if (saved_errno) { | |
| 639 __set_errno (saved_errno); | |
| 640 return (-1); | |
| 641 } | |
| 642 | |
| 643 return (0); | |
| 644 } | |
| 645 | |
|
11163
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
646 #if defined __linux__ \ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
647 && HAVE_SYS_VFS_H && HAVE_FSTATFS && HAVE_STRUCT_STATFS_F_TYPE |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
648 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
649 #include <sys/vfs.h> |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
650 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
651 /* Linux-specific constants from coreutils' src/fs.h */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
652 # define S_MAGIC_TMPFS 0x1021994 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
653 # define S_MAGIC_NFS 0x6969 |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
654 # define S_MAGIC_REISERFS 0x52654973 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
655 # define S_MAGIC_PROC 0x9FA0 |
|
11163
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
656 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
657 /* Return false if it is easy to determine the file system type of |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
658 the directory on which DIR_FD is open, and sorting dirents on |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
659 inode numbers is known not to improve traversal performance with |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
660 that type of file system. Otherwise, return true. */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
661 static bool |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
662 dirent_inode_sort_may_be_useful (int dir_fd) |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
663 { |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
664 /* Skip the sort only if we can determine efficiently |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
665 that skipping it is the right thing to do. |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
666 The cost of performing an unnecessary sort is negligible, |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
667 while the cost of *not* performing it can be O(N^2) with |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
668 a very large constant. */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
669 struct statfs fs_buf; |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
670 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
671 /* If fstatfs fails, assume sorting would be useful. */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
672 if (fstatfs (dir_fd, &fs_buf) != 0) |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
673 return true; |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
674 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
675 /* FIXME: what about when f_type is not an integral type? |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
676 deal with that if/when it's encountered. */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
677 switch (fs_buf.f_type) |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
678 { |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
679 case S_MAGIC_TMPFS: |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
680 case S_MAGIC_NFS: |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
681 /* On a file system of any of these types, sorting |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
682 is unnecessary, and hence wasteful. */ |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
683 return false; |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
684 |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
685 default: |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
686 return true; |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
687 } |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
688 } |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
689 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
690 /* Given a file descriptor DIR_FD open on a directory D, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
691 return true if it is valid to apply the leaf-optimization |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
692 technique of counting directories in D via stat.st_nlink. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
693 static bool |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
694 leaf_optimization_applies (int dir_fd) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
695 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
696 struct statfs fs_buf; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
697 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
698 /* If fstatfs fails, assume we can't use the optimization. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
699 if (fstatfs (dir_fd, &fs_buf) != 0) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
700 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
701 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
702 /* FIXME: do we need to detect AFS mount points? I doubt it, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
703 unless fstatfs can report S_MAGIC_REISERFS for such a directory. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
704 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
705 switch (fs_buf.f_type) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
706 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
707 /* List here the file system types that lack useable dirent.d_type |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
708 info, yet for which the optimization does apply. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
709 case S_MAGIC_REISERFS: |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
710 return true; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
711 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
712 case S_MAGIC_PROC: |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
713 /* Explicitly listing this or any other file system type for which |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
714 the optimization is not applicable is not necessary, but we leave |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
715 it here to document the risk. Per http://bugs.debian.org/143111, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
716 /proc may have bogus stat.st_nlink values. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
717 /* fall through */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
718 default: |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
719 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
720 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
721 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
722 |
|
11163
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
723 #else |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
724 static bool dirent_inode_sort_may_be_useful (int dir_fd) { return true; } |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
725 static bool leaf_optimization_applies (int dir_fd) { return false; } |
|
11163
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
726 #endif |
|
f448fc8fe832
fts: move a function definition "up" (no semantic change)
Jim Meyering <meyering@redhat.com>
parents:
11162
diff
changeset
|
727 |
|
11169
cc4b6d391fce
fts: add #if guards so that the fts_lgpl module still builds
Jim Meyering <meyering@redhat.com>
parents:
11164
diff
changeset
|
728 #if GNULIB_FTS |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
729 /* link-count-optimization entry: |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
730 map an stat.st_dev number to a boolean: leaf_optimization_works */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
731 struct LCO_ent |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
732 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
733 dev_t st_dev; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
734 bool opt_ok; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
735 }; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
736 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
737 /* Use a tiny initial size. If a traversal encounters more than |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
738 a few devices, the cost of growing/rehashing this table will be |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
739 rendered negligible by the number of inodes processed. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
740 enum { LCO_HT_INITIAL_SIZE = 13 }; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
741 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
742 static size_t |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
743 LCO_hash (void const *x, size_t table_size) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
744 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
745 struct LCO_ent const *ax = x; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
746 return (uintmax_t) ax->st_dev % table_size; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
747 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
748 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
749 static bool |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
750 LCO_compare (void const *x, void const *y) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
751 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
752 struct LCO_ent const *ax = x; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
753 struct LCO_ent const *ay = y; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
754 return ax->st_dev == ay->st_dev; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
755 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
756 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
757 /* Ask the same question as leaf_optimization_applies, but query |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
758 the cache first (FTS.fts_leaf_optimization_works_ht), and if necessary, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
759 update that cache. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
760 static bool |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
761 link_count_optimize_ok (FTSENT const *p) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
762 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
763 FTS *sp = p->fts_fts; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
764 Hash_table *h = sp->fts_leaf_optimization_works_ht; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
765 struct LCO_ent tmp; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
766 struct LCO_ent *ent; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
767 bool opt_ok; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
768 struct LCO_ent *t2; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
769 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
770 /* If we're not in CWDFD mode, don't bother with this optimization, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
771 since the caller is not serious about performance. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
772 if (!ISSET(FTS_CWDFD)) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
773 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
774 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
775 /* map st_dev to the boolean, leaf_optimization_works */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
776 if (h == NULL) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
777 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
778 h = sp->fts_leaf_optimization_works_ht |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
779 = hash_initialize (LCO_HT_INITIAL_SIZE, NULL, LCO_hash, |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
780 LCO_compare, free); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
781 if (h == NULL) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
782 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
783 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
784 tmp.st_dev = p->fts_statp->st_dev; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
785 ent = hash_lookup (h, &tmp); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
786 if (ent) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
787 return ent->opt_ok; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
788 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
789 /* Look-up failed. Query directly and cache the result. */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
790 t2 = malloc (sizeof *t2); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
791 if (t2 == NULL) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
792 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
793 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
794 /* Is it ok to perform the optimization in the dir, FTS_CWD_FD? */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
795 opt_ok = leaf_optimization_applies (sp->fts_cwd_fd); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
796 t2->opt_ok = opt_ok; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
797 t2->st_dev = p->fts_statp->st_dev; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
798 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
799 ent = hash_insert (h, t2); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
800 if (ent == NULL) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
801 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
802 /* insertion failed */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
803 free (t2); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
804 return false; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
805 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
806 fts_assert (ent == t2); |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
807 |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
808 return opt_ok; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
809 } |
|
11169
cc4b6d391fce
fts: add #if guards so that the fts_lgpl module still builds
Jim Meyering <meyering@redhat.com>
parents:
11164
diff
changeset
|
810 #endif |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
811 |
| 5858 | 812 /* |
| 5907 | 813 * Special case of "/" at the end of the file name so that slashes aren't |
| 814 * appended which would cause file names to be written as "....//foo". | |
| 5858 | 815 */ |
| 816 #define NAPPEND(p) \ | |
| 817 (p->fts_path[p->fts_pathlen - 1] == '/' \ | |
| 818 ? p->fts_pathlen - 1 : p->fts_pathlen) | |
| 819 | |
| 820 FTSENT * | |
| 821 fts_read (register FTS *sp) | |
| 822 { | |
| 823 register FTSENT *p, *tmp; | |
| 824 register unsigned short int instr; | |
| 825 register char *t; | |
| 826 | |
| 827 /* If finished or unrecoverable error, return NULL. */ | |
| 828 if (sp->fts_cur == NULL || ISSET(FTS_STOP)) | |
| 829 return (NULL); | |
| 830 | |
| 831 /* Set current node pointer. */ | |
| 832 p = sp->fts_cur; | |
| 833 | |
| 834 /* Save and zero out user instructions. */ | |
| 835 instr = p->fts_instr; | |
| 836 p->fts_instr = FTS_NOINSTR; | |
| 837 | |
| 838 /* Any type of file may be re-visited; re-stat and re-turn. */ | |
| 839 if (instr == FTS_AGAIN) { | |
| 840 p->fts_info = fts_stat(sp, p, false); | |
| 841 return (p); | |
| 842 } | |
| 843 Dprintf (("fts_read: p=%s\n", | |
| 844 p->fts_info == FTS_INIT ? "" : p->fts_path)); | |
| 845 | |
| 846 /* | |
| 847 * Following a symlink -- SLNONE test allows application to see | |
| 848 * SLNONE and recover. If indirecting through a symlink, have | |
| 849 * keep a pointer to current location. If unable to get that | |
| 850 * pointer, follow fails. | |
| 851 */ | |
| 852 if (instr == FTS_FOLLOW && | |
| 853 (p->fts_info == FTS_SL || p->fts_info == FTS_SLNONE)) { | |
| 854 p->fts_info = fts_stat(sp, p, true); | |
| 855 if (p->fts_info == FTS_D && !ISSET(FTS_NOCHDIR)) { | |
| 7151 | 856 if ((p->fts_symfd = diropen (sp, ".")) < 0) { |
| 5858 | 857 p->fts_errno = errno; |
| 858 p->fts_info = FTS_ERR; | |
| 859 } else | |
| 860 p->fts_flags |= FTS_SYMFOLLOW; | |
| 861 } | |
| 5872 | 862 goto check_for_dir; |
| 5858 | 863 } |
| 864 | |
| 865 /* Directory in pre-order. */ | |
| 866 if (p->fts_info == FTS_D) { | |
| 867 /* If skipped or crossed mount point, do post-order visit. */ | |
| 868 if (instr == FTS_SKIP || | |
| 869 (ISSET(FTS_XDEV) && p->fts_statp->st_dev != sp->fts_dev)) { | |
| 870 if (p->fts_flags & FTS_SYMFOLLOW) | |
| 871 (void)close(p->fts_symfd); | |
| 872 if (sp->fts_child) { | |
| 873 fts_lfree(sp->fts_child); | |
| 874 sp->fts_child = NULL; | |
| 875 } | |
| 876 p->fts_info = FTS_DP; | |
| 877 LEAVE_DIR (sp, p, "1"); | |
| 878 return (p); | |
| 879 } | |
| 880 | |
| 881 /* Rebuild if only read the names and now traversing. */ | |
| 882 if (sp->fts_child != NULL && ISSET(FTS_NAMEONLY)) { | |
| 883 CLR(FTS_NAMEONLY); | |
| 884 fts_lfree(sp->fts_child); | |
| 885 sp->fts_child = NULL; | |
| 886 } | |
| 887 | |
| 888 /* | |
| 889 * Cd to the subdirectory. | |
| 890 * | |
| 891 * If have already read and now fail to chdir, whack the list | |
| 892 * to make the names come out right, and set the parent errno | |
| 893 * so the application will eventually get an error condition. | |
| 894 * Set the FTS_DONTCHDIR flag so that when we logically change | |
| 895 * directories back to the parent we don't do a chdir. | |
| 896 * | |
| 897 * If haven't read do so. If the read fails, fts_build sets | |
| 898 * FTS_STOP or the fts_info field of the node. | |
| 899 */ | |
| 900 if (sp->fts_child != NULL) { | |
| 901 if (fts_safe_changedir(sp, p, -1, p->fts_accpath)) { | |
| 902 p->fts_errno = errno; | |
| 903 p->fts_flags |= FTS_DONTCHDIR; | |
| 904 for (p = sp->fts_child; p != NULL; | |
| 905 p = p->fts_link) | |
| 906 p->fts_accpath = | |
| 907 p->fts_parent->fts_accpath; | |
| 908 } | |
| 909 } else if ((sp->fts_child = fts_build(sp, BREAD)) == NULL) { | |
| 910 if (ISSET(FTS_STOP)) | |
| 911 return (NULL); | |
| 912 /* If fts_build's call to fts_safe_changedir failed | |
| 913 because it was not able to fchdir into a | |
| 914 subdirectory, tell the caller. */ | |
|
9129
79b0e6e0f7ba
* lib/fts.c (fts_read): Upon failure to chdir into a subdirectory,
Jim Meyering <jim@meyering.net>
parents:
7853
diff
changeset
|
915 if (p->fts_errno && p->fts_info != FTS_DNR) |
| 5858 | 916 p->fts_info = FTS_ERR; |
| 917 LEAVE_DIR (sp, p, "2"); | |
| 918 return (p); | |
| 919 } | |
| 920 p = sp->fts_child; | |
| 921 sp->fts_child = NULL; | |
| 922 goto name; | |
| 923 } | |
| 924 | |
| 925 /* Move to the next node on this level. */ | |
| 926 next: tmp = p; | |
| 927 if ((p = p->fts_link) != NULL) { | |
|
7853
9ddbddb3fc43
fts.c: a small readability/maintainability improvement
Jim Meyering <jim@meyering.net>
parents:
7797
diff
changeset
|
928 sp->fts_cur = p; |
| 5858 | 929 free(tmp); |
| 930 | |
| 931 /* | |
| 932 * If reached the top, return to the original directory (or | |
| 5907 | 933 * the root of the tree), and load the file names for the next |
| 934 * root. | |
| 5858 | 935 */ |
| 936 if (p->fts_level == FTS_ROOTLEVEL) { | |
| 7151 | 937 if (RESTORE_INITIAL_CWD(sp)) { |
| 5858 | 938 SET(FTS_STOP); |
| 939 return (NULL); | |
| 940 } | |
|
11722
92457e8bc574
fts: avoid false-positive cycle-detection
Jim Meyering <meyering@redhat.com>
parents:
11171
diff
changeset
|
941 free_dir(sp); |
| 5858 | 942 fts_load(sp, p); |
|
11722
92457e8bc574
fts: avoid false-positive cycle-detection
Jim Meyering <meyering@redhat.com>
parents:
11171
diff
changeset
|
943 setup_dir(sp); |
| 5872 | 944 goto check_for_dir; |
| 5858 | 945 } |
| 946 | |
| 947 /* | |
| 948 * User may have called fts_set on the node. If skipped, | |
| 949 * ignore. If followed, get a file descriptor so we can | |
| 950 * get back if necessary. | |
| 951 */ | |
| 952 if (p->fts_instr == FTS_SKIP) | |
| 953 goto next; | |
| 954 if (p->fts_instr == FTS_FOLLOW) { | |
| 955 p->fts_info = fts_stat(sp, p, true); | |
| 956 if (p->fts_info == FTS_D && !ISSET(FTS_NOCHDIR)) { | |
| 7151 | 957 if ((p->fts_symfd = diropen (sp, ".")) < 0) { |
| 5858 | 958 p->fts_errno = errno; |
| 959 p->fts_info = FTS_ERR; | |
| 960 } else | |
| 961 p->fts_flags |= FTS_SYMFOLLOW; | |
| 962 } | |
| 963 p->fts_instr = FTS_NOINSTR; | |
| 964 } | |
| 965 | |
| 966 name: t = sp->fts_path + NAPPEND(p->fts_parent); | |
| 967 *t++ = '/'; | |
| 968 memmove(t, p->fts_name, p->fts_namelen + 1); | |
| 5872 | 969 check_for_dir: |
|
7853
9ddbddb3fc43
fts.c: a small readability/maintainability improvement
Jim Meyering <jim@meyering.net>
parents:
7797
diff
changeset
|
970 sp->fts_cur = p; |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
971 if (p->fts_info == FTS_NSOK) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
972 { |
|
7786
f79f76e0b6f7
Use a more robust test for a "can't happen" condition.
Jim Meyering <jim@meyering.net>
parents:
7785
diff
changeset
|
973 if (p->fts_statp->st_size == FTS_STAT_REQUIRED) |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
974 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
975 FTSENT *parent = p->fts_parent; |
|
11171
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
976 if (FTS_ROOTLEVEL < p->fts_level |
|
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
977 /* ->fts_n_dirs_remaining is not valid |
|
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
978 for command-line-specified names. */ |
|
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
979 && parent->fts_n_dirs_remaining == 0 |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
980 && ISSET(FTS_NOSTAT) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
981 && ISSET(FTS_PHYSICAL) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
982 && link_count_optimize_ok (parent)) |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
983 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
984 /* nothing more needed */ |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
985 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
986 else |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
987 { |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
988 p->fts_info = fts_stat(sp, p, false); |
|
11171
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
989 if (S_ISDIR(p->fts_statp->st_mode) |
|
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
990 && p->fts_level != FTS_ROOTLEVEL |
|
5be124843f38
fts: avoid used-uninitialized error due to recent change
Jim Meyering <meyering@redhat.com>
parents:
11169
diff
changeset
|
991 && parent->fts_n_dirs_remaining) |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
992 parent->fts_n_dirs_remaining--; |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
993 } |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
994 } |
|
7786
f79f76e0b6f7
Use a more robust test for a "can't happen" condition.
Jim Meyering <jim@meyering.net>
parents:
7785
diff
changeset
|
995 else |
|
f79f76e0b6f7
Use a more robust test for a "can't happen" condition.
Jim Meyering <jim@meyering.net>
parents:
7785
diff
changeset
|
996 fts_assert (p->fts_statp->st_size == FTS_NO_STAT_REQUIRED); |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
997 } |
|
7707
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
998 |
| 5858 | 999 if (p->fts_info == FTS_D) |
| 5872 | 1000 { |
|
7707
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
1001 /* Now that P->fts_statp is guaranteed to be valid, |
|
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
1002 if this is a command-line directory, record its |
|
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
1003 device number, to be used for FTS_XDEV. */ |
|
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
1004 if (p->fts_level == FTS_ROOTLEVEL) |
|
3f57a32346a2
* lib/fts.c (fts_load): Don't set sp->fts_dev here, since
Jim Meyering <jim@meyering.net>
parents:
7684
diff
changeset
|
1005 sp->fts_dev = p->fts_statp->st_dev; |
|
7635
7f65b3a0e7a4
* lib/fts.c [FTS_DEBUG]: Don't try to print a pointer via %s.
Jim Meyering <jim@meyering.net>
parents:
7623
diff
changeset
|
1006 Dprintf ((" entering: %s\n", p->fts_path)); |
| 5872 | 1007 if (! enter_dir (sp, p)) |
| 1008 { | |
| 1009 __set_errno (ENOMEM); | |
| 1010 return NULL; | |
| 1011 } | |
| 1012 } | |
| 1013 return p; | |
| 5858 | 1014 } |
| 1015 | |
| 1016 /* Move up to the parent node. */ | |
| 1017 p = tmp->fts_parent; | |
|
7853
9ddbddb3fc43
fts.c: a small readability/maintainability improvement
Jim Meyering <jim@meyering.net>
parents:
7797
diff
changeset
|
1018 sp->fts_cur = p; |
| 5858 | 1019 free(tmp); |
| 1020 | |
| 1021 if (p->fts_level == FTS_ROOTPARENTLEVEL) { | |
| 1022 /* | |
| 1023 * Done; free everything up and set errno to 0 so the user | |
| 1024 * can distinguish between error and EOF. | |
| 1025 */ | |
| 1026 free(p); | |
| 1027 __set_errno (0); | |
| 1028 return (sp->fts_cur = NULL); | |
| 1029 } | |
| 1030 | |
|
7785
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
1031 fts_assert (p->fts_info != FTS_NSOK); |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1032 |
| 5907 | 1033 /* NUL terminate the file name. */ |
| 5858 | 1034 sp->fts_path[p->fts_pathlen] = '\0'; |
| 1035 | |
| 1036 /* | |
| 7151 | 1037 * Return to the parent directory. If at a root node, restore |
| 1038 * the initial working directory. If we came through a symlink, | |
| 1039 * go back through the file descriptor. Otherwise, move up | |
| 1040 * one level, via "..". | |
| 5858 | 1041 */ |
| 1042 if (p->fts_level == FTS_ROOTLEVEL) { | |
| 7151 | 1043 if (RESTORE_INITIAL_CWD(sp)) { |
| 5858 | 1044 p->fts_errno = errno; |
| 1045 SET(FTS_STOP); | |
| 1046 } | |
| 1047 } else if (p->fts_flags & FTS_SYMFOLLOW) { | |
| 1048 if (FCHDIR(sp, p->fts_symfd)) { | |
| 7151 | 1049 int saved_errno = errno; |
| 5858 | 1050 (void)close(p->fts_symfd); |
| 1051 __set_errno (saved_errno); | |
| 1052 p->fts_errno = errno; | |
| 1053 SET(FTS_STOP); | |
| 1054 } | |
| 1055 (void)close(p->fts_symfd); | |
| 1056 } else if (!(p->fts_flags & FTS_DONTCHDIR) && | |
| 1057 fts_safe_changedir(sp, p->fts_parent, -1, "..")) { | |
| 1058 p->fts_errno = errno; | |
| 1059 SET(FTS_STOP); | |
| 1060 } | |
| 1061 p->fts_info = p->fts_errno ? FTS_ERR : FTS_DP; | |
| 1062 if (p->fts_errno == 0) | |
| 1063 LEAVE_DIR (sp, p, "3"); | |
| 1064 return ISSET(FTS_STOP) ? NULL : p; | |
| 1065 } | |
| 1066 | |
| 1067 /* | |
| 1068 * Fts_set takes the stream as an argument although it's not used in this | |
| 1069 * implementation; it would be necessary if anyone wanted to add global | |
| 1070 * semantics to fts using fts_set. An error return is allowed for similar | |
| 1071 * reasons. | |
| 1072 */ | |
| 1073 /* ARGSUSED */ | |
| 1074 int | |
| 1075 fts_set(FTS *sp ATTRIBUTE_UNUSED, FTSENT *p, int instr) | |
| 1076 { | |
| 1077 if (instr != 0 && instr != FTS_AGAIN && instr != FTS_FOLLOW && | |
| 1078 instr != FTS_NOINSTR && instr != FTS_SKIP) { | |
| 1079 __set_errno (EINVAL); | |
| 1080 return (1); | |
| 1081 } | |
| 1082 p->fts_instr = instr; | |
| 1083 return (0); | |
| 1084 } | |
| 1085 | |
| 1086 FTSENT * | |
| 1087 fts_children (register FTS *sp, int instr) | |
| 1088 { | |
| 1089 register FTSENT *p; | |
| 1090 int fd; | |
| 1091 | |
| 1092 if (instr != 0 && instr != FTS_NAMEONLY) { | |
| 1093 __set_errno (EINVAL); | |
| 1094 return (NULL); | |
| 1095 } | |
| 1096 | |
| 1097 /* Set current node pointer. */ | |
| 1098 p = sp->fts_cur; | |
| 1099 | |
| 1100 /* | |
| 1101 * Errno set to 0 so user can distinguish empty directory from | |
| 1102 * an error. | |
| 1103 */ | |
| 1104 __set_errno (0); | |
| 1105 | |
| 1106 /* Fatal errors stop here. */ | |
| 1107 if (ISSET(FTS_STOP)) | |
| 1108 return (NULL); | |
| 1109 | |
| 1110 /* Return logical hierarchy of user's arguments. */ | |
| 1111 if (p->fts_info == FTS_INIT) | |
| 1112 return (p->fts_link); | |
| 1113 | |
| 1114 /* | |
| 1115 * If not a directory being visited in pre-order, stop here. Could | |
| 1116 * allow FTS_DNR, assuming the user has fixed the problem, but the | |
| 1117 * same effect is available with FTS_AGAIN. | |
| 1118 */ | |
| 1119 if (p->fts_info != FTS_D /* && p->fts_info != FTS_DNR */) | |
| 1120 return (NULL); | |
| 1121 | |
| 1122 /* Free up any previous child list. */ | |
| 1123 if (sp->fts_child != NULL) | |
| 1124 fts_lfree(sp->fts_child); | |
| 1125 | |
| 1126 if (instr == FTS_NAMEONLY) { | |
| 1127 SET(FTS_NAMEONLY); | |
| 1128 instr = BNAMES; | |
| 1129 } else | |
| 1130 instr = BCHILD; | |
| 1131 | |
| 1132 /* | |
| 5907 | 1133 * If using chdir on a relative file name and called BEFORE fts_read |
| 1134 * does its chdir to the root of a traversal, we can lose -- we need to | |
| 5858 | 1135 * chdir into the subdirectory, and we don't know where the current |
| 1136 * directory is, so we can't get back so that the upcoming chdir by | |
| 1137 * fts_read will work. | |
| 1138 */ | |
| 1139 if (p->fts_level != FTS_ROOTLEVEL || p->fts_accpath[0] == '/' || | |
| 1140 ISSET(FTS_NOCHDIR)) | |
| 1141 return (sp->fts_child = fts_build(sp, instr)); | |
| 1142 | |
| 7151 | 1143 if ((fd = diropen (sp, ".")) < 0) |
| 5858 | 1144 return (sp->fts_child = NULL); |
| 1145 sp->fts_child = fts_build(sp, instr); | |
| 7151 | 1146 if (ISSET(FTS_CWDFD)) |
| 1147 { | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1148 cwd_advance_fd (sp, fd, true); |
| 7151 | 1149 } |
| 1150 else | |
| 1151 { | |
| 1152 if (fchdir(fd)) | |
| 1153 { | |
|
6611
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
1154 int saved_errno = errno; |
| 7151 | 1155 close (fd); |
|
6611
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
1156 __set_errno (saved_errno); |
| 7151 | 1157 return NULL; |
| 1158 } | |
| 1159 close (fd); | |
| 1160 } | |
| 5858 | 1161 return (sp->fts_child); |
| 1162 } | |
| 1163 | |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1164 /* A comparison function to sort on increasing inode number. |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1165 For some file system types, sorting either way makes a huge |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1166 performance difference for a directory with very many entries, |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1167 but sorting on increasing values is slightly better than sorting |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1168 on decreasing values. The difference is in the 5% range. */ |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1169 static int |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1170 fts_compare_ino (struct _ftsent const **a, struct _ftsent const **b) |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1171 { |
|
10482
e8d1a5b2c49c
fts: tweak inode comparison function
Jim Meyering <meyering@redhat.com>
parents:
10481
diff
changeset
|
1172 return (a[0]->fts_statp->st_ino < b[0]->fts_statp->st_ino ? -1 |
|
e8d1a5b2c49c
fts: tweak inode comparison function
Jim Meyering <meyering@redhat.com>
parents:
10481
diff
changeset
|
1173 : b[0]->fts_statp->st_ino < a[0]->fts_statp->st_ino ? 1 : 0); |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1174 } |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1175 |
|
10843
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1176 /* Map the dirent.d_type value, DTYPE, to the corresponding stat.st_mode |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1177 S_IF* bit and set ST.st_mode, thus clearing all other bits in that field. */ |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1178 static void |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1179 set_stat_type (struct stat *st, unsigned int dtype) |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1180 { |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1181 mode_t type; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1182 switch (dtype) |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1183 { |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1184 case DT_BLK: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1185 type = S_IFBLK; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1186 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1187 case DT_CHR: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1188 type = S_IFCHR; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1189 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1190 case DT_DIR: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1191 type = S_IFDIR; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1192 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1193 case DT_FIFO: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1194 type = S_IFIFO; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1195 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1196 case DT_LNK: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1197 type = S_IFLNK; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1198 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1199 case DT_REG: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1200 type = S_IFREG; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1201 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1202 case DT_SOCK: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1203 type = S_IFSOCK; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1204 break; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1205 default: |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1206 type = 0; |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1207 } |
| 10844 | 1208 st->st_mode = type; |
|
10843
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1209 } |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1210 |
| 5858 | 1211 /* |
| 1212 * This is the tricky part -- do not casually change *anything* in here. The | |
| 1213 * idea is to build the linked list of entries that are used by fts_children | |
| 1214 * and fts_read. There are lots of special cases. | |
| 1215 * | |
| 1216 * The real slowdown in walking the tree is the stat calls. If FTS_NOSTAT is | |
| 1217 * set and it's a physical walk (so that symbolic links can't be directories), | |
| 1218 * we can do things quickly. First, if it's a 4.4BSD file system, the type | |
| 1219 * of the file is in the directory entry. Otherwise, we assume that the number | |
| 1220 * of subdirectories in a node is equal to the number of links to the parent. | |
| 1221 * The former skips all stat calls. The latter skips stat calls in any leaf | |
| 1222 * directories and for any files after the subdirectories in the directory have | |
| 1223 * been found, cutting the stat calls by about 2/3. | |
| 1224 */ | |
| 1225 static FTSENT * | |
| 1226 internal_function | |
| 1227 fts_build (register FTS *sp, int type) | |
| 1228 { | |
| 1229 register struct dirent *dp; | |
| 1230 register FTSENT *p, *head; | |
| 1231 register size_t nitems; | |
| 1232 FTSENT *cur, *tail; | |
| 1233 DIR *dirp; | |
| 1234 void *oldaddr; | |
| 1235 int saved_errno; | |
| 1236 bool descend; | |
| 1237 bool doadjust; | |
| 1238 ptrdiff_t level; | |
| 1239 nlink_t nlinks; | |
| 1240 bool nostat; | |
| 1241 size_t len, maxlen, new_len; | |
| 1242 char *cp; | |
| 1243 | |
| 1244 /* Set current node pointer. */ | |
| 1245 cur = sp->fts_cur; | |
| 1246 | |
| 1247 /* | |
| 1248 * Open the directory for reading. If this fails, we're done. | |
| 1249 * If being called from fts_read, set the fts_info field. | |
| 1250 */ | |
| 1251 #if defined FTS_WHITEOUT && 0 | |
| 1252 if (ISSET(FTS_WHITEOUT)) | |
| 1253 oflag = DTF_NODUP|DTF_REWIND; | |
| 1254 else | |
| 1255 oflag = DTF_HIDEW|DTF_NODUP|DTF_REWIND; | |
| 1256 #else | |
| 7151 | 1257 # define __opendir2(file, flag) \ |
| 1258 ( ! ISSET(FTS_NOCHDIR) && ISSET(FTS_CWDFD) \ | |
| 1259 ? opendirat(sp->fts_cwd_fd, file) \ | |
| 1260 : opendir(file)) | |
| 5858 | 1261 #endif |
| 1262 if ((dirp = __opendir2(cur->fts_accpath, oflag)) == NULL) { | |
| 1263 if (type == BREAD) { | |
| 1264 cur->fts_info = FTS_DNR; | |
| 1265 cur->fts_errno = errno; | |
| 1266 } | |
| 1267 return (NULL); | |
| 1268 } | |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1269 /* Rather than calling fts_stat for each and every entry encountered |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1270 in the readdir loop (below), stat each directory only right after |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1271 opening it. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1272 if (cur->fts_info == FTS_NSOK) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1273 cur->fts_info = fts_stat(sp, cur, false); |
| 5858 | 1274 |
| 1275 /* | |
| 1276 * Nlinks is the number of possible entries of type directory in the | |
| 1277 * directory if we're cheating on stat calls, 0 if we're not doing | |
| 1278 * any stat calls at all, (nlink_t) -1 if we're statting everything. | |
| 1279 */ | |
| 1280 if (type == BNAMES) { | |
| 1281 nlinks = 0; | |
| 1282 /* Be quiet about nostat, GCC. */ | |
| 1283 nostat = false; | |
| 1284 } else if (ISSET(FTS_NOSTAT) && ISSET(FTS_PHYSICAL)) { | |
| 1285 nlinks = (cur->fts_statp->st_nlink | |
| 1286 - (ISSET(FTS_SEEDOT) ? 0 : 2)); | |
| 1287 nostat = true; | |
| 1288 } else { | |
| 1289 nlinks = -1; | |
| 1290 nostat = false; | |
| 1291 } | |
| 1292 | |
| 1293 /* | |
| 1294 * If we're going to need to stat anything or we want to descend | |
| 1295 * and stay in the directory, chdir. If this fails we keep going, | |
| 1296 * but set a flag so we don't chdir after the post-order visit. | |
| 1297 * We won't be able to stat anything, but we can still return the | |
| 1298 * names themselves. Note, that since fts_read won't be able to | |
| 5907 | 1299 * chdir into the directory, it will have to return different file |
| 5858 | 1300 * names than before, i.e. "a/b" instead of "b". Since the node |
| 1301 * has already been visited in pre-order, have to wait until the | |
| 1302 * post-order visit to return the error. There is a special case | |
| 1303 * here, if there was nothing to stat then it's not an error to | |
| 1304 * not be able to stat. This is all fairly nasty. If a program | |
| 1305 * needed sorted entries or stat information, they had better be | |
| 1306 * checking FTS_NS on the returned nodes. | |
| 1307 */ | |
| 1308 if (nlinks || type == BREAD) { | |
| 7151 | 1309 int dir_fd = dirfd(dirp); |
| 1310 if (ISSET(FTS_CWDFD) && 0 <= dir_fd) | |
| 1311 dir_fd = dup (dir_fd); | |
| 1312 if (dir_fd < 0 || fts_safe_changedir(sp, cur, dir_fd, NULL)) { | |
| 5858 | 1313 if (nlinks && type == BREAD) |
| 1314 cur->fts_errno = errno; | |
| 1315 cur->fts_flags |= FTS_DONTCHDIR; | |
| 1316 descend = false; | |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
1317 closedir(dirp); |
| 7151 | 1318 if (ISSET(FTS_CWDFD) && 0 <= dir_fd) |
| 1319 close (dir_fd); | |
| 5858 | 1320 dirp = NULL; |
| 1321 } else | |
| 1322 descend = true; | |
| 1323 } else | |
| 1324 descend = false; | |
| 1325 | |
| 1326 /* | |
| 1327 * Figure out the max file name length that can be stored in the | |
| 5907 | 1328 * current buffer -- the inner loop allocates more space as necessary. |
| 5858 | 1329 * We really wouldn't have to do the maxlen calculations here, we |
| 5907 | 1330 * could do them in fts_read before returning the name, but it's a |
| 5858 | 1331 * lot easier here since the length is part of the dirent structure. |
| 1332 * | |
| 1333 * If not changing directories set a pointer so that can just append | |
| 5907 | 1334 * each new component into the file name. |
| 5858 | 1335 */ |
| 1336 len = NAPPEND(cur); | |
| 1337 if (ISSET(FTS_NOCHDIR)) { | |
| 1338 cp = sp->fts_path + len; | |
| 1339 *cp++ = '/'; | |
| 1340 } else { | |
| 1341 /* GCC, you're too verbose. */ | |
| 1342 cp = NULL; | |
| 1343 } | |
| 1344 len++; | |
| 1345 maxlen = sp->fts_pathlen - len; | |
| 1346 | |
| 1347 level = cur->fts_level + 1; | |
| 1348 | |
| 1349 /* Read the directory, attaching each entry to the `link' pointer. */ | |
| 1350 doadjust = false; | |
| 1351 for (head = tail = NULL, nitems = 0; dirp && (dp = readdir(dirp));) { | |
|
7555
0da62e3fa6c7
(fts_build): Move variable declaration, for C89 compliance.
Bruno Haible <bruno@clisp.org>
parents:
7499
diff
changeset
|
1352 bool is_dir; |
|
0da62e3fa6c7
(fts_build): Move variable declaration, for C89 compliance.
Bruno Haible <bruno@clisp.org>
parents:
7499
diff
changeset
|
1353 |
| 5858 | 1354 if (!ISSET(FTS_SEEDOT) && ISDOT(dp->d_name)) |
| 1355 continue; | |
| 1356 | |
|
6949
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1357 if ((p = fts_alloc (sp, dp->d_name, |
|
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1358 _D_EXACT_NAMLEN (dp))) == NULL) |
| 5858 | 1359 goto mem1; |
|
6949
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1360 if (_D_EXACT_NAMLEN (dp) >= maxlen) { |
|
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1361 /* include space for NUL */ |
| 5858 | 1362 oldaddr = sp->fts_path; |
|
6949
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1363 if (! fts_palloc(sp, _D_EXACT_NAMLEN (dp) + len + 1)) { |
| 5858 | 1364 /* |
| 5907 | 1365 * No more memory. Save |
| 5858 | 1366 * errno, free up the current structure and the |
| 1367 * structures already allocated. | |
| 1368 */ | |
| 1369 mem1: saved_errno = errno; | |
|
7396
6f79ccad009d
2006-10-02 Dmitry V. Levin <ldv@altlinux.org>
Jim Meyering <jim@meyering.net>
parents:
7302
diff
changeset
|
1370 free(p); |
| 5858 | 1371 fts_lfree(head); |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
1372 closedir(dirp); |
| 5858 | 1373 cur->fts_info = FTS_ERR; |
| 1374 SET(FTS_STOP); | |
| 1375 __set_errno (saved_errno); | |
| 1376 return (NULL); | |
| 1377 } | |
| 1378 /* Did realloc() change the pointer? */ | |
| 1379 if (oldaddr != sp->fts_path) { | |
| 1380 doadjust = true; | |
| 1381 if (ISSET(FTS_NOCHDIR)) | |
| 1382 cp = sp->fts_path + len; | |
| 1383 } | |
| 1384 maxlen = sp->fts_pathlen - len; | |
| 1385 } | |
| 1386 | |
|
6949
96af1224c6fa
* backupfile.c, dirfd.h, fts.c, getcwd.c, glob.c, glob_.h:
Paul Eggert <eggert@cs.ucla.edu>
parents:
6934
diff
changeset
|
1387 new_len = len + _D_EXACT_NAMLEN (dp); |
| 5858 | 1388 if (new_len < len) { |
| 1389 /* | |
|
9718
5954dae30c7f
* lib/fts.c (fts_build): Typo correction to comment.
Martin Buchholz <martin@xemacs.org>
parents:
9613
diff
changeset
|
1390 * In the unlikely event that we would end up |
| 5907 | 1391 * with a file name longer than SIZE_MAX, free up |
| 5858 | 1392 * the current structure and the structures already |
| 1393 * allocated, then error out with ENAMETOOLONG. | |
| 1394 */ | |
| 1395 free(p); | |
| 1396 fts_lfree(head); | |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
1397 closedir(dirp); |
| 5858 | 1398 cur->fts_info = FTS_ERR; |
| 1399 SET(FTS_STOP); | |
| 1400 __set_errno (ENAMETOOLONG); | |
| 1401 return (NULL); | |
| 1402 } | |
| 1403 p->fts_level = level; | |
| 1404 p->fts_parent = sp->fts_cur; | |
| 1405 p->fts_pathlen = new_len; | |
| 1406 | |
| 1407 #if defined FTS_WHITEOUT && 0 | |
| 1408 if (dp->d_type == DT_WHT) | |
| 1409 p->fts_flags |= FTS_ISW; | |
| 1410 #endif | |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1411 /* Store dirent.d_ino, in case we need to sort |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1412 entries before processing them. */ |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1413 p->fts_statp->st_ino = D_INO (dp); |
| 5858 | 1414 |
| 7151 | 1415 /* Build a file name for fts_stat to stat. */ |
| 1416 if (ISSET(FTS_NOCHDIR)) { | |
| 1417 p->fts_accpath = p->fts_path; | |
| 1418 memmove(cp, p->fts_name, p->fts_namelen + 1); | |
| 1419 } else | |
| 1420 p->fts_accpath = p->fts_name; | |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1421 |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1422 if (sp->fts_compar == NULL || ISSET(FTS_DEFER_STAT)) { |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1423 /* Record what fts_read will have to do with this |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1424 entry. In many cases, it will simply fts_stat it, |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1425 but we can take advantage of any d_type information |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1426 to optimize away the unnecessary stat calls. I.e., |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1427 if FTS_NOSTAT is in effect and we're not following |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1428 symlinks (FTS_PHYSICAL) and d_type indicates this |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1429 is *not* a directory, then we won't have to stat it |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1430 at all. If it *is* a directory, then (currently) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1431 we stat it regardless, in order to get device and |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1432 inode numbers. Some day we might optimize that |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1433 away, too, for directories where d_ino is known to |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1434 be valid. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1435 bool skip_stat = (ISSET(FTS_PHYSICAL) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1436 && ISSET(FTS_NOSTAT) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1437 && DT_IS_KNOWN(dp) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1438 && ! DT_MUST_BE(dp, DT_DIR)); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1439 p->fts_info = FTS_NSOK; |
|
10843
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1440 /* Propagate dirent.d_type information back |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1441 to caller, when possible. */ |
|
36479bd141ee
fts: provide dirent.d_type via FTSENT.fts_statp, when possible
Jim Meyering <meyering@redhat.com>
parents:
10688
diff
changeset
|
1442 set_stat_type (p->fts_statp, D_TYPE (dp)); |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1443 fts_set_stat_required(p, !skip_stat); |
|
11162
15e1c0e6fd35
fts: correct internal computation of nlinks (optimization-related)
Jim Meyering <meyering@redhat.com>
parents:
10863
diff
changeset
|
1444 is_dir = (ISSET(FTS_PHYSICAL) |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1445 && DT_MUST_BE(dp, DT_DIR)); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1446 } else { |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1447 p->fts_info = fts_stat(sp, p, false); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1448 is_dir = (p->fts_info == FTS_D |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1449 || p->fts_info == FTS_DC |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1450 || p->fts_info == FTS_DOT); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1451 } |
| 5858 | 1452 |
| 7151 | 1453 /* Decrement link count if applicable. */ |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1454 if (nlinks > 0 && is_dir) |
| 7151 | 1455 nlinks -= nostat; |
| 5858 | 1456 |
| 1457 /* We walk in directory order so "ls -f" doesn't get upset. */ | |
| 1458 p->fts_link = NULL; | |
| 1459 if (head == NULL) | |
| 1460 head = tail = p; | |
| 1461 else { | |
| 1462 tail->fts_link = p; | |
| 1463 tail = p; | |
| 1464 } | |
| 1465 ++nitems; | |
| 1466 } | |
| 1467 if (dirp) | |
|
5867
67b499052f7f
* fts.c: Include fts_.h first, to check interface.
Paul Eggert <eggert@cs.ucla.edu>
parents:
5858
diff
changeset
|
1468 closedir(dirp); |
| 5858 | 1469 |
| 1470 /* | |
| 5907 | 1471 * If realloc() changed the address of the file name, adjust the |
| 5858 | 1472 * addresses for the rest of the tree and the dir list. |
| 1473 */ | |
| 1474 if (doadjust) | |
| 1475 fts_padjust(sp, head); | |
| 1476 | |
| 1477 /* | |
| 5907 | 1478 * If not changing directories, reset the file name back to original |
| 5858 | 1479 * state. |
| 1480 */ | |
| 1481 if (ISSET(FTS_NOCHDIR)) { | |
| 1482 if (len == sp->fts_pathlen || nitems == 0) | |
| 1483 --cp; | |
| 1484 *cp = '\0'; | |
| 1485 } | |
| 1486 | |
| 1487 /* | |
| 1488 * If descended after called from fts_children or after called from | |
| 1489 * fts_read and nothing found, get back. At the root level we use | |
| 5907 | 1490 * the saved fd; if one of fts_open()'s arguments is a relative name |
| 5858 | 1491 * to an empty directory, we wind up here with no other way back. If |
| 1492 * can't get back, we're done. | |
| 1493 */ | |
| 1494 if (descend && (type == BCHILD || !nitems) && | |
| 7151 | 1495 (cur->fts_level == FTS_ROOTLEVEL |
| 1496 ? RESTORE_INITIAL_CWD(sp) | |
| 1497 : fts_safe_changedir(sp, cur->fts_parent, -1, ".."))) { | |
| 5858 | 1498 cur->fts_info = FTS_ERR; |
| 1499 SET(FTS_STOP); | |
| 7151 | 1500 fts_lfree(head); |
| 5858 | 1501 return (NULL); |
| 1502 } | |
| 1503 | |
| 1504 /* If didn't find anything, return NULL. */ | |
| 1505 if (!nitems) { | |
| 1506 if (type == BREAD) | |
| 1507 cur->fts_info = FTS_DP; | |
| 7151 | 1508 fts_lfree(head); |
| 5858 | 1509 return (NULL); |
| 1510 } | |
| 1511 | |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1512 /* If there are many entries, no sorting function has been specified, |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1513 and this file system is of a type that may be slow with a large |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1514 number of entries, then sort the directory entries on increasing |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1515 inode numbers. */ |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1516 if (nitems > _FTS_INODE_SORT_DIR_ENTRIES_THRESHOLD |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1517 && !sp->fts_compar |
|
10561
5e359fc738d5
fts.c: adjust a new interface to be more generally useful
Jim Meyering <meyering@redhat.com>
parents:
10482
diff
changeset
|
1518 && ISSET (FTS_CWDFD) |
|
5e359fc738d5
fts.c: adjust a new interface to be more generally useful
Jim Meyering <meyering@redhat.com>
parents:
10482
diff
changeset
|
1519 && dirent_inode_sort_may_be_useful (sp->fts_cwd_fd)) { |
|
10481
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1520 sp->fts_compar = fts_compare_ino; |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1521 head = fts_sort (sp, head, nitems); |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1522 sp->fts_compar = NULL; |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1523 } |
|
47fe4e48e158
fts: sort dirent entries on inode number before traversing
Jim Meyering <meyering@redhat.com>
parents:
9718
diff
changeset
|
1524 |
| 5858 | 1525 /* Sort the entries. */ |
| 1526 if (sp->fts_compar && nitems > 1) | |
| 1527 head = fts_sort(sp, head, nitems); | |
| 1528 return (head); | |
| 1529 } | |
| 1530 | |
| 1531 #if FTS_DEBUG | |
| 1532 | |
| 1533 /* Walk ->fts_parent links starting at E_CURR, until the root of the | |
| 1534 current hierarchy. There should be a directory with dev/inode | |
| 1535 matching those of AD. If not, print a lot of diagnostics. */ | |
| 1536 static void | |
| 1537 find_matching_ancestor (FTSENT const *e_curr, struct Active_dir const *ad) | |
| 1538 { | |
| 1539 FTSENT const *ent; | |
| 1540 for (ent = e_curr; ent->fts_level >= FTS_ROOTLEVEL; ent = ent->fts_parent) | |
| 1541 { | |
| 1542 if (ad->ino == ent->fts_statp->st_ino | |
| 1543 && ad->dev == ent->fts_statp->st_dev) | |
| 1544 return; | |
| 1545 } | |
| 1546 printf ("ERROR: tree dir, %s, not active\n", ad->fts_ent->fts_accpath); | |
| 1547 printf ("active dirs:\n"); | |
| 1548 for (ent = e_curr; | |
| 1549 ent->fts_level >= FTS_ROOTLEVEL; ent = ent->fts_parent) | |
| 1550 printf (" %s(%"PRIuMAX"/%"PRIuMAX") to %s(%"PRIuMAX"/%"PRIuMAX")...\n", | |
| 1551 ad->fts_ent->fts_accpath, | |
| 1552 (uintmax_t) ad->dev, | |
| 1553 (uintmax_t) ad->ino, | |
| 1554 ent->fts_accpath, | |
| 1555 (uintmax_t) ent->fts_statp->st_dev, | |
| 1556 (uintmax_t) ent->fts_statp->st_ino); | |
| 1557 } | |
| 1558 | |
| 1559 void | |
| 1560 fts_cross_check (FTS const *sp) | |
| 1561 { | |
| 1562 FTSENT const *ent = sp->fts_cur; | |
| 1563 FTSENT const *t; | |
| 1564 if ( ! ISSET (FTS_TIGHT_CYCLE_CHECK)) | |
| 1565 return; | |
| 1566 | |
| 1567 Dprintf (("fts-cross-check cur=%s\n", ent->fts_path)); | |
| 1568 /* Make sure every parent dir is in the tree. */ | |
| 1569 for (t = ent->fts_parent; t->fts_level >= FTS_ROOTLEVEL; t = t->fts_parent) | |
| 1570 { | |
| 1571 struct Active_dir ad; | |
| 1572 ad.ino = t->fts_statp->st_ino; | |
| 1573 ad.dev = t->fts_statp->st_dev; | |
|
6034
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
1574 if ( ! hash_lookup (sp->fts_cycle.ht, &ad)) |
| 5858 | 1575 printf ("ERROR: active dir, %s, not in tree\n", t->fts_path); |
| 1576 } | |
| 1577 | |
| 1578 /* Make sure every dir in the tree is an active dir. | |
| 1579 But ENT is not necessarily a directory. If so, just skip this part. */ | |
| 1580 if (ent->fts_parent->fts_level >= FTS_ROOTLEVEL | |
| 1581 && (ent->fts_info == FTS_DP | |
| 1582 || ent->fts_info == FTS_D)) | |
| 1583 { | |
| 1584 struct Active_dir *ad; | |
|
6034
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
1585 for (ad = hash_get_first (sp->fts_cycle.ht); ad != NULL; |
|
96149b1bbb32
(fts_cross_check) [FTS_DEBUG]: s/active_dir_ht/fts_cycle.ht/.
Jim Meyering <jim@meyering.net>
parents:
5907
diff
changeset
|
1586 ad = hash_get_next (sp->fts_cycle.ht, ad)) |
| 5858 | 1587 { |
| 1588 find_matching_ancestor (ent, ad); | |
| 1589 } | |
| 1590 } | |
| 1591 } | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1592 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1593 static bool |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1594 same_fd (int fd1, int fd2) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1595 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1596 struct stat sb1, sb2; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1597 return (fstat (fd1, &sb1) == 0 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1598 && fstat (fd2, &sb2) == 0 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1599 && SAME_INODE (sb1, sb2)); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1600 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1601 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1602 static void |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1603 fd_ring_print (FTS const *sp, FILE *stream, char const *msg) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1604 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1605 I_ring const *fd_ring = &sp->fts_fd_ring; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1606 unsigned int i = fd_ring->fts_front; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1607 char *cwd = getcwdat (sp->fts_cwd_fd, NULL, 0); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1608 fprintf (stream, "=== %s ========== %s\n", msg, cwd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1609 free (cwd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1610 if (i_ring_empty (fd_ring)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1611 return; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1612 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1613 while (true) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1614 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1615 int fd = fd_ring->fts_fd_ring[i]; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1616 if (fd < 0) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1617 fprintf (stream, "%d: %d:\n", i, fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1618 else |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1619 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1620 char *wd = getcwdat (fd, NULL, 0); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1621 fprintf (stream, "%d: %d: %s\n", i, fd, wd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1622 free (wd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1623 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1624 if (i == fd_ring->fts_back) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1625 break; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1626 i = (i + I_RING_SIZE - 1) % I_RING_SIZE; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1627 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1628 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1629 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1630 /* Ensure that each file descriptor on the fd_ring matches a |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1631 parent, grandparent, etc. of the current working directory. */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1632 static void |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1633 fd_ring_check (FTS const *sp) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1634 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1635 if (!fts_debug) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1636 return; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1637 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1638 /* Make a writable copy. */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1639 I_ring fd_w = sp->fts_fd_ring; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1640 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1641 int cwd_fd = sp->fts_cwd_fd; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1642 cwd_fd = dup (cwd_fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1643 char *dot = getcwdat (cwd_fd, NULL, 0); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1644 error (0, 0, "===== check ===== cwd: %s", dot); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1645 free (dot); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1646 while ( ! i_ring_empty (&fd_w)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1647 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1648 int fd = i_ring_pop (&fd_w); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1649 if (0 <= fd) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1650 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1651 int parent_fd = openat (cwd_fd, "..", O_RDONLY); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1652 if (parent_fd < 0) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1653 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1654 // Warn? |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1655 break; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1656 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1657 if (!same_fd (fd, parent_fd)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1658 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1659 char *cwd = getcwdat (fd, NULL, 0); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1660 error (0, errno, "ring : %s", cwd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1661 char *c2 = getcwdat (parent_fd, NULL, 0); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1662 error (0, errno, "parent: %s", c2); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1663 free (cwd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1664 free (c2); |
|
7785
3246679850d5
Slight readability improvement: use an assert-like macro
Jim Meyering <jim@meyering.net>
parents:
7707
diff
changeset
|
1665 fts_assert (0); |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1666 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1667 close (cwd_fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1668 cwd_fd = parent_fd; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1669 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1670 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1671 close (cwd_fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1672 } |
| 5858 | 1673 #endif |
| 1674 | |
| 1675 static unsigned short int | |
| 1676 internal_function | |
| 1677 fts_stat(FTS *sp, register FTSENT *p, bool follow) | |
| 1678 { | |
| 1679 struct stat *sbp = p->fts_statp; | |
| 1680 int saved_errno; | |
| 1681 | |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1682 if (p->fts_level == FTS_ROOTLEVEL && ISSET(FTS_COMFOLLOW)) |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1683 follow = true; |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1684 |
| 5858 | 1685 #if defined FTS_WHITEOUT && 0 |
| 1686 /* check for whiteout */ | |
| 1687 if (p->fts_flags & FTS_ISW) { | |
| 1688 memset(sbp, '\0', sizeof (*sbp)); | |
| 1689 sbp->st_mode = S_IFWHT; | |
| 1690 return (FTS_W); | |
| 1691 } | |
| 1692 #endif | |
| 1693 | |
| 1694 /* | |
| 1695 * If doing a logical walk, or application requested FTS_FOLLOW, do | |
| 1696 * a stat(2). If that fails, check for a non-existent symlink. If | |
| 1697 * fail, set the errno from the stat call. | |
| 1698 */ | |
| 1699 if (ISSET(FTS_LOGICAL) || follow) { | |
| 1700 if (stat(p->fts_accpath, sbp)) { | |
| 1701 saved_errno = errno; | |
|
6611
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
1702 if (errno == ENOENT |
|
e4456c51dde7
Sync from the stable (b5) branch of coreutils:
Jim Meyering <jim@meyering.net>
parents:
6540
diff
changeset
|
1703 && lstat(p->fts_accpath, sbp) == 0) { |
| 5858 | 1704 __set_errno (0); |
| 1705 return (FTS_SLNONE); | |
| 1706 } | |
| 1707 p->fts_errno = saved_errno; | |
| 1708 goto err; | |
| 1709 } | |
| 7151 | 1710 } else if (fstatat(sp->fts_cwd_fd, p->fts_accpath, sbp, |
| 1711 AT_SYMLINK_NOFOLLOW)) { | |
| 5858 | 1712 p->fts_errno = errno; |
| 1713 err: memset(sbp, 0, sizeof(struct stat)); | |
| 1714 return (FTS_NS); | |
| 1715 } | |
| 1716 | |
| 1717 if (S_ISDIR(sbp->st_mode)) { | |
|
11164
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
1718 p->fts_n_dirs_remaining = (sbp->st_nlink |
|
90715da41753
fts: arrange not to stat non-directories in more cases
Jim Meyering <meyering@redhat.com>
parents:
11163
diff
changeset
|
1719 - (ISSET(FTS_SEEDOT) ? 0 : 2)); |
|
7482
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1720 if (ISDOT(p->fts_name)) { |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1721 /* Command-line "." and ".." are real directories. */ |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1722 return (p->fts_level == FTS_ROOTLEVEL ? FTS_D : FTS_DOT); |
|
26914593f079
Big performance improvement for fts-based tools that use FTS_NOSTAT.
Jim Meyering <jim@meyering.net>
parents:
7408
diff
changeset
|
1723 } |
| 5872 | 1724 |
|
7797
10432bdf90d3
2007-01-08 Bruno Haible <bruno@clisp.org>
Bruno Haible <bruno@clisp.org>
parents:
7786
diff
changeset
|
1725 #if !GNULIB_FTS |
| 5872 | 1726 { |
| 1727 /* | |
| 1728 * Cycle detection is done by brute force when the directory | |
| 1729 * is first encountered. If the tree gets deep enough or the | |
| 1730 * number of symbolic links to directories is high enough, | |
| 1731 * something faster might be worthwhile. | |
| 1732 */ | |
| 1733 FTSENT *t; | |
| 1734 | |
| 1735 for (t = p->fts_parent; | |
| 1736 t->fts_level >= FTS_ROOTLEVEL; t = t->fts_parent) | |
| 1737 if (sbp->st_ino == t->fts_statp->st_ino | |
| 1738 && sbp->st_dev == t->fts_statp->st_dev) | |
| 1739 { | |
| 1740 p->fts_cycle = t; | |
| 1741 return (FTS_DC); | |
| 1742 } | |
| 1743 } | |
| 1744 #endif | |
| 1745 | |
| 5858 | 1746 return (FTS_D); |
| 1747 } | |
| 1748 if (S_ISLNK(sbp->st_mode)) | |
| 1749 return (FTS_SL); | |
| 1750 if (S_ISREG(sbp->st_mode)) | |
| 1751 return (FTS_F); | |
| 1752 return (FTS_DEFAULT); | |
| 1753 } | |
| 1754 | |
| 1755 static int | |
| 1756 fts_compar (void const *a, void const *b) | |
| 1757 { | |
| 1758 /* Convert A and B to the correct types, to pacify the compiler, and | |
| 1759 for portability to bizarre hosts where "void const *" and "FTSENT | |
| 1760 const **" differ in runtime representation. The comparison | |
| 1761 function cannot modify *a and *b, but there is no compile-time | |
| 1762 check for this. */ | |
| 1763 FTSENT const **pa = (FTSENT const **) a; | |
| 1764 FTSENT const **pb = (FTSENT const **) b; | |
| 1765 return pa[0]->fts_fts->fts_compar (pa, pb); | |
| 1766 } | |
| 1767 | |
| 1768 static FTSENT * | |
| 1769 internal_function | |
| 1770 fts_sort (FTS *sp, FTSENT *head, register size_t nitems) | |
| 1771 { | |
| 1772 register FTSENT **ap, *p; | |
| 1773 | |
| 1774 /* On most modern hosts, void * and FTSENT ** have the same | |
| 1775 run-time representation, and one can convert sp->fts_compar to | |
| 1776 the type qsort expects without problem. Use the heuristic that | |
| 1777 this is OK if the two pointer types are the same size, and if | |
| 1778 converting FTSENT ** to long int is the same as converting | |
| 1779 FTSENT ** to void * and then to long int. This heuristic isn't | |
| 1780 valid in general but we don't know of any counterexamples. */ | |
| 1781 FTSENT *dummy; | |
| 1782 int (*compare) (void const *, void const *) = | |
| 1783 ((sizeof &dummy == sizeof (void *) | |
| 1784 && (long int) &dummy == (long int) (void *) &dummy) | |
| 1785 ? (int (*) (void const *, void const *)) sp->fts_compar | |
| 1786 : fts_compar); | |
| 1787 | |
| 1788 /* | |
| 1789 * Construct an array of pointers to the structures and call qsort(3). | |
| 1790 * Reassemble the array in the order returned by qsort. If unable to | |
| 1791 * sort for memory reasons, return the directory entries in their | |
| 1792 * current order. Allocate enough space for the current needs plus | |
| 1793 * 40 so don't realloc one entry at a time. | |
| 1794 */ | |
| 1795 if (nitems > sp->fts_nitems) { | |
|
9562
1850c982220c
Avoid use of private FTS type name.
Jim Meyering <jim@meyering.net>
parents:
9309
diff
changeset
|
1796 FTSENT **a; |
| 5858 | 1797 |
| 1798 sp->fts_nitems = nitems + 40; | |
| 1799 if (SIZE_MAX / sizeof *a < sp->fts_nitems | |
| 1800 || ! (a = realloc (sp->fts_array, | |
| 1801 sp->fts_nitems * sizeof *a))) { | |
| 1802 free(sp->fts_array); | |
| 1803 sp->fts_array = NULL; | |
| 1804 sp->fts_nitems = 0; | |
| 1805 return (head); | |
| 1806 } | |
| 1807 sp->fts_array = a; | |
| 1808 } | |
| 1809 for (ap = sp->fts_array, p = head; p; p = p->fts_link) | |
| 1810 *ap++ = p; | |
| 1811 qsort((void *)sp->fts_array, nitems, sizeof(FTSENT *), compare); | |
| 1812 for (head = *(ap = sp->fts_array); --nitems; ++ap) | |
| 1813 ap[0]->fts_link = ap[1]; | |
| 1814 ap[0]->fts_link = NULL; | |
| 1815 return (head); | |
| 1816 } | |
| 1817 | |
| 1818 static FTSENT * | |
| 1819 internal_function | |
| 1820 fts_alloc (FTS *sp, const char *name, register size_t namelen) | |
| 1821 { | |
| 1822 register FTSENT *p; | |
| 1823 size_t len; | |
| 1824 | |
| 1825 /* | |
| 1826 * The file name is a variable length array. Allocate the FTSENT | |
| 1827 * structure and the file name in one chunk. | |
| 1828 */ | |
| 1829 len = sizeof(FTSENT) + namelen; | |
| 1830 if ((p = malloc(len)) == NULL) | |
| 1831 return (NULL); | |
| 1832 | |
| 1833 /* Copy the name and guarantee NUL termination. */ | |
| 1834 memmove(p->fts_name, name, namelen); | |
| 1835 p->fts_name[namelen] = '\0'; | |
| 1836 | |
| 1837 p->fts_namelen = namelen; | |
| 1838 p->fts_fts = sp; | |
| 1839 p->fts_path = sp->fts_path; | |
| 1840 p->fts_errno = 0; | |
| 1841 p->fts_flags = 0; | |
| 1842 p->fts_instr = FTS_NOINSTR; | |
| 1843 p->fts_number = 0; | |
| 1844 p->fts_pointer = NULL; | |
| 1845 return (p); | |
| 1846 } | |
| 1847 | |
| 1848 static void | |
| 1849 internal_function | |
| 1850 fts_lfree (register FTSENT *head) | |
| 1851 { | |
| 1852 register FTSENT *p; | |
| 1853 | |
| 1854 /* Free a linked list of structures. */ | |
| 1855 while ((p = head)) { | |
| 1856 head = head->fts_link; | |
| 1857 free(p); | |
| 1858 } | |
| 1859 } | |
| 1860 | |
| 1861 /* | |
| 5907 | 1862 * Allow essentially unlimited file name lengths; find, rm, ls should |
| 1863 * all work on any tree. Most systems will allow creation of file | |
| 1864 * names much longer than MAXPATHLEN, even though the kernel won't | |
| 1865 * resolve them. Add the size (not just what's needed) plus 256 bytes | |
| 1866 * so don't realloc the file name 2 bytes at a time. | |
| 5858 | 1867 */ |
| 1868 static bool | |
| 1869 internal_function | |
| 1870 fts_palloc (FTS *sp, size_t more) | |
| 1871 { | |
| 1872 char *p; | |
| 1873 size_t new_len = sp->fts_pathlen + more + 256; | |
| 1874 | |
| 1875 /* | |
| 1876 * See if fts_pathlen would overflow. | |
| 1877 */ | |
| 1878 if (new_len < sp->fts_pathlen) { | |
|
7396
6f79ccad009d
2006-10-02 Dmitry V. Levin <ldv@altlinux.org>
Jim Meyering <jim@meyering.net>
parents:
7302
diff
changeset
|
1879 free(sp->fts_path); |
| 5858 | 1880 sp->fts_path = NULL; |
| 1881 __set_errno (ENAMETOOLONG); | |
| 1882 return false; | |
| 1883 } | |
| 1884 sp->fts_pathlen = new_len; | |
| 1885 p = realloc(sp->fts_path, sp->fts_pathlen); | |
| 1886 if (p == NULL) { | |
| 1887 free(sp->fts_path); | |
| 1888 sp->fts_path = NULL; | |
| 1889 return false; | |
| 1890 } | |
| 1891 sp->fts_path = p; | |
| 1892 return true; | |
| 1893 } | |
| 1894 | |
| 1895 /* | |
| 5907 | 1896 * When the file name is realloc'd, have to fix all of the pointers in |
| 1897 * structures already returned. | |
| 5858 | 1898 */ |
| 1899 static void | |
| 1900 internal_function | |
| 1901 fts_padjust (FTS *sp, FTSENT *head) | |
| 1902 { | |
| 1903 FTSENT *p; | |
| 1904 char *addr = sp->fts_path; | |
| 1905 | |
| 1906 #define ADJUST(p) do { \ | |
| 1907 if ((p)->fts_accpath != (p)->fts_name) { \ | |
| 1908 (p)->fts_accpath = \ | |
| 1909 (char *)addr + ((p)->fts_accpath - (p)->fts_path); \ | |
| 1910 } \ | |
| 1911 (p)->fts_path = addr; \ | |
| 1912 } while (0) | |
| 1913 /* Adjust the current set of children. */ | |
| 1914 for (p = sp->fts_child; p; p = p->fts_link) | |
| 1915 ADJUST(p); | |
| 1916 | |
| 1917 /* Adjust the rest of the tree, including the current level. */ | |
| 1918 for (p = head; p->fts_level >= FTS_ROOTLEVEL;) { | |
| 1919 ADJUST(p); | |
| 1920 p = p->fts_link ? p->fts_link : p->fts_parent; | |
| 1921 } | |
| 1922 } | |
| 1923 | |
| 1924 static size_t | |
| 1925 internal_function | |
| 1926 fts_maxarglen (char * const *argv) | |
| 1927 { | |
| 1928 size_t len, max; | |
| 1929 | |
| 1930 for (max = 0; *argv; ++argv) | |
| 1931 if ((len = strlen(*argv)) > max) | |
| 1932 max = len; | |
| 1933 return (max + 1); | |
| 1934 } | |
| 1935 | |
| 1936 /* | |
| 5907 | 1937 * Change to dir specified by fd or file name without getting |
| 5858 | 1938 * tricked by someone changing the world out from underneath us. |
| 1939 * Assumes p->fts_statp->st_dev and p->fts_statp->st_ino are filled in. | |
| 7151 | 1940 * If FD is non-negative, expect it to be used after this function returns, |
| 1941 * and to be closed eventually. So don't pass e.g., `dirfd(dirp)' and then | |
| 1942 * do closedir(dirp), because that would invalidate the saved FD. | |
| 1943 * Upon failure, close FD immediately and return nonzero. | |
| 5858 | 1944 */ |
| 1945 static int | |
| 1946 internal_function | |
| 5907 | 1947 fts_safe_changedir (FTS *sp, FTSENT *p, int fd, char const *dir) |
| 5858 | 1948 { |
| 7151 | 1949 int ret; |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1950 bool is_dotdot = dir && STREQ (dir, ".."); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1951 int newfd; |
|
7623
5bfad7d666d6
* lib/fts.c (fts_safe_changedir): Add a comment.
Jim Meyering <jim@meyering.net>
parents:
7621
diff
changeset
|
1952 |
|
5bfad7d666d6
* lib/fts.c (fts_safe_changedir): Add a comment.
Jim Meyering <jim@meyering.net>
parents:
7621
diff
changeset
|
1953 /* This clause handles the unusual case in which FTS_NOCHDIR |
|
5bfad7d666d6
* lib/fts.c (fts_safe_changedir): Add a comment.
Jim Meyering <jim@meyering.net>
parents:
7621
diff
changeset
|
1954 is specified, along with FTS_CWDFD. In that case, there is |
|
5bfad7d666d6
* lib/fts.c (fts_safe_changedir): Add a comment.
Jim Meyering <jim@meyering.net>
parents:
7621
diff
changeset
|
1955 no need to change even the virtual cwd file descriptor. |
|
5bfad7d666d6
* lib/fts.c (fts_safe_changedir): Add a comment.
Jim Meyering <jim@meyering.net>
parents:
7621
diff
changeset
|
1956 However, if FD is non-negative, we do close it here. */ |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1957 if (ISSET (FTS_NOCHDIR)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1958 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1959 if (ISSET (FTS_CWDFD) && 0 <= fd) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1960 close (fd); |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1961 return 0; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1962 } |
| 5858 | 1963 |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1964 if (fd < 0 && is_dotdot && ISSET (FTS_CWDFD)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1965 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1966 /* When possible, skip the diropen and subsequent fstat+dev/ino |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1967 comparison. I.e., when changing to parent directory |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1968 (chdir ("..")), use a file descriptor from the ring and |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1969 save the overhead of diropen+fstat, as well as avoiding |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1970 failure when we lack "x" access to the virtual cwd. */ |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1971 if ( ! i_ring_empty (&sp->fts_fd_ring)) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1972 { |
|
7681
a6b7b31e9ddf
* lib/fts.c (fts_safe_changedir): Move a declaration "up",
Jim Meyering <jim@meyering.net>
parents:
7639
diff
changeset
|
1973 int parent_fd; |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1974 fd_ring_print (sp, stderr, "pre-pop"); |
|
7681
a6b7b31e9ddf
* lib/fts.c (fts_safe_changedir): Move a declaration "up",
Jim Meyering <jim@meyering.net>
parents:
7639
diff
changeset
|
1975 parent_fd = i_ring_pop (&sp->fts_fd_ring); |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1976 is_dotdot = true; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1977 if (0 <= parent_fd) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1978 { |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1979 fd = parent_fd; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1980 dir = NULL; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1981 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1982 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1983 } |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1984 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1985 newfd = fd; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1986 if (fd < 0 && (newfd = diropen (sp, dir)) < 0) |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1987 return -1; |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1988 |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1989 /* The following dev/inode check is necessary if we're doing a |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1990 `logical' traversal (through symlinks, a la chown -L), if the |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1991 system lacks O_NOFOLLOW support, or if we're changing to ".." |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1992 (but not via a popped file descriptor). When changing to the |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1993 name "..", O_NOFOLLOW can't help. In general, when the target is |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1994 not "..", diropen's use of O_NOFOLLOW ensures we don't mistakenly |
|
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
1995 follow a symlink, so we can avoid the expense of this fstat. */ |
| 7408 | 1996 if (ISSET(FTS_LOGICAL) || ! HAVE_WORKING_O_NOFOLLOW |
| 7151 | 1997 || (dir && STREQ (dir, ".."))) |
| 1998 { | |
| 1999 struct stat sb; | |
| 2000 if (fstat(newfd, &sb)) | |
| 2001 { | |
| 5858 | 2002 ret = -1; |
| 2003 goto bail; | |
| 7151 | 2004 } |
| 2005 if (p->fts_statp->st_dev != sb.st_dev | |
| 2006 || p->fts_statp->st_ino != sb.st_ino) | |
| 2007 { | |
| 5858 | 2008 __set_errno (ENOENT); /* disinformation */ |
| 2009 ret = -1; | |
| 2010 goto bail; | |
| 7151 | 2011 } |
| 2012 } | |
| 2013 | |
| 2014 if (ISSET(FTS_CWDFD)) | |
| 2015 { | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
2016 cwd_advance_fd (sp, newfd, ! is_dotdot); |
| 7151 | 2017 return 0; |
| 2018 } | |
| 2019 | |
| 5858 | 2020 ret = fchdir(newfd); |
| 2021 bail: | |
| 2022 if (fd < 0) | |
| 7151 | 2023 { |
| 2024 int oerrno = errno; | |
| 2025 (void)close(newfd); | |
| 2026 __set_errno (oerrno); | |
| 2027 } | |
|
7639
1670d42131d7
Make fts (in FTS_CWDFD mode) more efficient by caching a few open
Jim Meyering <jim@meyering.net>
parents:
7635
diff
changeset
|
2028 return ret; |
| 5858 | 2029 } |