Tue Apr 14 05:21:26 2015 UTC ()
Pull up following revision(s) (requested by christos in ticket #691):
	lib/libc/gen/opendir.c: revision 1.39
- Use O_DIRECTORY to open the file, so that we don't need to stat() after
  that.
- Move the stat() call to fdopendir() and change it's error handling so that
  it does not hide errors.
- According to POSIX, fdopendir() transfers ownership of the fd only on
  success, so don't close it on failure. XXX: We still make it non-blocking
  on failure, but that's nitpicking.


(snj)
diff -r1.38 -r1.38.20.1 src/lib/libc/gen/opendir.c

cvs diff -r1.38 -r1.38.20.1 src/lib/libc/gen/opendir.c (switch to unified diff)

--- src/lib/libc/gen/opendir.c 2011/10/15 23:00:01 1.38
+++ src/lib/libc/gen/opendir.c 2015/04/14 05:21:26 1.38.20.1
@@ -1,164 +1,176 @@ @@ -1,164 +1,176 @@
1/* $NetBSD: opendir.c,v 1.38 2011/10/15 23:00:01 christos Exp $ */ 1/* $NetBSD: opendir.c,v 1.38.20.1 2015/04/14 05:21:26 snj Exp $ */
2 2
3/* 3/*
4 * Copyright (c) 1983, 1993 4 * Copyright (c) 1983, 1993
5 * The Regents of the University of California. All rights reserved. 5 * The Regents of the University of California. All rights reserved.
6 * 6 *
7 * Redistribution and use in source and binary forms, with or without 7 * Redistribution and use in source and binary forms, with or without
8 * modification, are permitted provided that the following conditions 8 * modification, are permitted provided that the following conditions
9 * are met: 9 * are met:
10 * 1. Redistributions of source code must retain the above copyright 10 * 1. Redistributions of source code must retain the above copyright
11 * notice, this list of conditions and the following disclaimer. 11 * notice, this list of conditions and the following disclaimer.
12 * 2. Redistributions in binary form must reproduce the above copyright 12 * 2. Redistributions in binary form must reproduce the above copyright
13 * notice, this list of conditions and the following disclaimer in the 13 * notice, this list of conditions and the following disclaimer in the
14 * documentation and/or other materials provided with the distribution. 14 * documentation and/or other materials provided with the distribution.
15 * 3. Neither the name of the University nor the names of its contributors 15 * 3. Neither the name of the University nor the names of its contributors
16 * may be used to endorse or promote products derived from this software 16 * may be used to endorse or promote products derived from this software
17 * without specific prior written permission. 17 * without specific prior written permission.
18 * 18 *
19 * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND 19 * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
20 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE 20 * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
21 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE 21 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
22 * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE 22 * ARE DISCLAIMED. IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
23 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL 23 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
24 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS 24 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
25 * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) 25 * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
26 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT 26 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
27 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY 27 * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
28 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF 28 * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
29 * SUCH DAMAGE. 29 * SUCH DAMAGE.
30 */ 30 */
31 31
32#include <sys/cdefs.h> 32#include <sys/cdefs.h>
33#if defined(LIBC_SCCS) && !defined(lint) 33#if defined(LIBC_SCCS) && !defined(lint)
34#if 0 34#if 0
35static char sccsid[] = "@(#)opendir.c 8.7 (Berkeley) 12/10/94"; 35static char sccsid[] = "@(#)opendir.c 8.7 (Berkeley) 12/10/94";
36#else 36#else
37__RCSID("$NetBSD: opendir.c,v 1.38 2011/10/15 23:00:01 christos Exp $"); 37__RCSID("$NetBSD: opendir.c,v 1.38.20.1 2015/04/14 05:21:26 snj Exp $");
38#endif 38#endif
39#endif /* LIBC_SCCS and not lint */ 39#endif /* LIBC_SCCS and not lint */
40 40
41#include "namespace.h" 41#include "namespace.h"
42#include "reentrant.h" 42#include "reentrant.h"
43#include "extern.h" 43#include "extern.h"
44 44
45#include <sys/param.h> 45#include <sys/param.h>
46#include <sys/mount.h> 46#include <sys/mount.h>
47#include <sys/stat.h> 47#include <sys/stat.h>
48 48
49#include <assert.h> 49#include <assert.h>
50#include <dirent.h> 50#include <dirent.h>
51#include <errno.h> 51#include <errno.h>
52#include <fcntl.h> 52#include <fcntl.h>
53#include <stdlib.h> 53#include <stdlib.h>
54#include <string.h> 54#include <string.h>
55#include <unistd.h> 55#include <unistd.h>
56 56
57#include "dirent_private.h" 57#include "dirent_private.h"
58 58
59static DIR *__opendir_common(int, const char *, int); 59static DIR *__opendir_common(int, const char *, int);
60 60
61__weak_alias(fdopendir,_fdopendir) 61__weak_alias(fdopendir,_fdopendir)
62 62
63/* 63/*
64 * Open a directory. 64 * Open a directory.
65 */ 65 */
66DIR * 66DIR *
67opendir(const char *name) 67opendir(const char *name)
68{ 68{
69 69
70 _DIAGASSERT(name != NULL); 70 _DIAGASSERT(name != NULL);
71 71
72 return (__opendir2(name, DTF_HIDEW|DTF_NODUP)); 72 return (__opendir2(name, DTF_HIDEW|DTF_NODUP));
73} 73}
74 74
75DIR * 75DIR *
76__opendir2(const char *name, int flags) 76__opendir2(const char *name, int flags)
77{ 77{
 78 DIR *dirp;
78 int fd; 79 int fd;
79 80
80 if ((fd = open(name, O_RDONLY | O_NONBLOCK | O_CLOEXEC)) == -1) 81 if ((fd = open(name, O_RDONLY|O_DIRECTORY|O_NONBLOCK|O_CLOEXEC)) == -1)
81 return NULL; 82 return NULL;
82 return __opendir_common(fd, name, flags); 83
 84 dirp = __opendir_common(fd, name, flags);
 85 if (dirp == NULL) {
 86 int serrno = errno;
 87 (void)close(fd);
 88 errno = serrno;
 89 }
 90 return dirp;
83} 91}
84 92
85#ifndef __LIBC12_SOURCE__ 93#ifndef __LIBC12_SOURCE__
86DIR * 94DIR *
87_fdopendir(int fd) 95_fdopendir(int fd)
88{ 96{
 97 struct stat sb;
 98
 99 if (fstat(fd, &sb) == -1)
 100 return NULL;
 101
 102 if (!S_ISDIR(sb.st_mode)) {
 103 errno = ENOTDIR;
 104 return NULL;
 105 }
 106
 107 /* This is optional according to POSIX, but a good measure */
89 if (fcntl(fd, F_SETFD, FD_CLOEXEC) == -1) 108 if (fcntl(fd, F_SETFD, FD_CLOEXEC) == -1)
90 return NULL; 109 return NULL;
91 110
92 return __opendir_common(fd, NULL, DTF_HIDEW|DTF_NODUP); 111 return __opendir_common(fd, NULL, DTF_HIDEW|DTF_NODUP);
93} 112}
94#endif 113#endif
95 114
96static DIR * 115static DIR *
97__opendir_common(int fd, const char *name, int flags) 116__opendir_common(int fd, const char *name, int flags)
98{ 117{
99 DIR *dirp = NULL; 118 DIR *dirp;
100 int serrno; 119 int serrno;
101 struct stat sb; 
102 struct statvfs sfb; 120 struct statvfs sfb;
103 int error; 121 int error;
104 122
105 if (fstat(fd, &sb) || !S_ISDIR(sb.st_mode)) { 
106 errno = ENOTDIR; 
107 goto error; 
108 } 
109 if ((dirp = malloc(sizeof(*dirp))) == NULL) 123 if ((dirp = malloc(sizeof(*dirp))) == NULL)
110 goto error; 124 goto error;
111 dirp->dd_buf = NULL; 125 dirp->dd_buf = NULL;
112 dirp->dd_internal = NULL; 126 dirp->dd_internal = NULL;
113#ifdef _REENTRANT 127#ifdef _REENTRANT
114 if (__isthreaded) { 128 if (__isthreaded) {
115 if ((dirp->dd_lock = malloc(sizeof(mutex_t))) == NULL) 129 if ((dirp->dd_lock = malloc(sizeof(mutex_t))) == NULL)
116 goto error; 130 goto error;
117 mutex_init((mutex_t *)dirp->dd_lock, NULL); 131 mutex_init((mutex_t *)dirp->dd_lock, NULL);
118 } 132 }
119#endif 133#endif
120 134
121 /* 135 /*
122 * Tweak flags for the underlying filesystem. 136 * Tweak flags for the underlying filesystem.
123 */ 137 */
124 138
125 if (fstatvfs1(fd, &sfb, ST_NOWAIT) < 0) 139 if (fstatvfs1(fd, &sfb, ST_NOWAIT) < 0)
126 goto error; 140 goto error;
127 if ((flags & DTF_NODUP) != 0) { 141 if ((flags & DTF_NODUP) != 0) {
128 if (!strncmp(sfb.f_fstypename, MOUNT_UNION, 142 if (!strncmp(sfb.f_fstypename, MOUNT_UNION,
129 sizeof(sfb.f_fstypename)) || 143 sizeof(sfb.f_fstypename)) ||
130 (sfb.f_flag & MNT_UNION) != 0) { 144 (sfb.f_flag & MNT_UNION) != 0) {
131 flags |= __DTF_READALL; 145 flags |= __DTF_READALL;
132 } else { 146 } else {
133 flags &= ~DTF_NODUP; 147 flags &= ~DTF_NODUP;
134 } 148 }
135 } 149 }
136 if (!strncmp(sfb.f_fstypename, MOUNT_NFS, sizeof(sfb.f_fstypename))) { 150 if (!strncmp(sfb.f_fstypename, MOUNT_NFS, sizeof(sfb.f_fstypename))) {
137 flags |= __DTF_READALL | __DTF_RETRY_ON_BADCOOKIE; 151 flags |= __DTF_READALL | __DTF_RETRY_ON_BADCOOKIE;
138 } 152 }
139 153
140 dirp->dd_flags = flags; 154 dirp->dd_flags = flags;
141 error = _initdir(dirp, fd, name); 155 error = _initdir(dirp, fd, name);
142 if (error) { 156 if (error) {
143 errno = error; 157 errno = error;
144 goto error; 158 goto error;
145 } 159 }
146 160
147 return (dirp); 161 return (dirp);
148error: 162error:
149 serrno = errno; 163 serrno = errno;
150 if (dirp != NULL) { 164 if (dirp != NULL) {
151#ifdef _REENTRANT 165#ifdef _REENTRANT
152 if (__isthreaded) { 166 if (__isthreaded) {
153 mutex_destroy((mutex_t *)dirp->dd_lock); 167 mutex_destroy((mutex_t *)dirp->dd_lock);
154 free(dirp->dd_lock); 168 free(dirp->dd_lock);
155 } 169 }
156#endif 170#endif
157 free(dirp->dd_buf); 171 free(dirp->dd_buf);
158 } 172 }
159 free(dirp); 173 free(dirp);
160 if (fd != -1) 
161 (void)close(fd); 
162 errno = serrno; 174 errno = serrno;
163 return NULL; 175 return NULL;
164} 176}