annotate tests/test-dirent-safer.c @ 17415:6550127da196

argp: typo fix * lib/argp-help.c: Typo in comment.
author Alexandre Duret-Lutz <adl@lrde.epita.fr>
date Fri, 17 May 2013 19:01:14 +0200
parents e542fd46ad6f
children
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
1 /* Test that directory streams leave standard fds alone.
17249
e542fd46ad6f maint: update all copyright year number ranges
Eric Blake <eblake@redhat.com>
parents: 16201
diff changeset
2 Copyright (C) 2009-2013 Free Software Foundation, Inc.
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
3
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
4 This program is free software: you can redistribute it and/or modify
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
5 it under the terms of the GNU General Public License as published by
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
6 the Free Software Foundation; either version 3 of the License, or
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
7 (at your option) any later version.
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
8
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
9 This program is distributed in the hope that it will be useful,
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
10 but WITHOUT ANY WARRANTY; without even the implied warranty of
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
11 MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
12 GNU General Public License for more details.
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
13
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
14 You should have received a copy of the GNU General Public License
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
15 along with this program. If not, see <http://www.gnu.org/licenses/>. */
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
16
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
17 /* Written by Eric Blake <ebb9@byu.net>, 2009. */
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
18
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
19 #include <config.h>
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
20
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
21 #include "dirent--.h"
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
22
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
23 #include <errno.h>
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
24 #include <fcntl.h>
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
25 #include <stdio.h>
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
26 #include <unistd.h>
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
27
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
28 #include "unistd-safer.h"
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
29
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
30 /* This test intentionally closes stderr. So, we arrange to have fd 10
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
31 (outside the range of interesting fd's during the test) set up to
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
32 duplicate the original stderr. */
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
33
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
34 #define BACKUP_STDERR_FILENO 10
12510
618282a72d9b tests: use macros.h in more places
Eric Blake <ebb9@byu.net>
parents: 12197
diff changeset
35 #define ASSERT_STREAM myerr
618282a72d9b tests: use macros.h in more places
Eric Blake <ebb9@byu.net>
parents: 12197
diff changeset
36 #include "macros.h"
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
37
12510
618282a72d9b tests: use macros.h in more places
Eric Blake <ebb9@byu.net>
parents: 12197
diff changeset
38 static FILE *myerr;
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
39
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
40 int
12197
e45d9bb2233e tests: avoid several compiler warnings
Eric Blake <ebb9@byu.net>
parents: 11936
diff changeset
41 main (void)
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
42 {
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
43 int i;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
44 DIR *dp;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
45 /* The dirent-safer module works without the use of fdopendir (which
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
46 would also pull in fchdir and openat); but if those modules were
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
47 also used, we ensure that they are safe. In particular, the
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
48 gnulib version of fdopendir is unable to guarantee that
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
49 dirfd(fdopendir(fd))==fd, but we can at least guarantee that if
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
50 they are not equal, the fd returned by dirfd is safe. */
13073
03de1efbfdd3 Resolve conflict between the two kinds of module indicators.
Bruno Haible <bruno@clisp.org>
parents: 12559
diff changeset
51 #if HAVE_FDOPENDIR || GNULIB_TEST_FDOPENDIR
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
52 int dfd;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
53 #endif
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
54
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
55 /* We close fd 2 later, so save it in fd 10. */
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
56 if (dup2 (STDERR_FILENO, BACKUP_STDERR_FILENO) != BACKUP_STDERR_FILENO
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
57 || (myerr = fdopen (BACKUP_STDERR_FILENO, "w")) == NULL)
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
58 return 2;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
59
13073
03de1efbfdd3 Resolve conflict between the two kinds of module indicators.
Bruno Haible <bruno@clisp.org>
parents: 12559
diff changeset
60 #if HAVE_FDOPENDIR || GNULIB_TEST_FDOPENDIR
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
61 dfd = open (".", O_RDONLY);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
62 ASSERT (STDERR_FILENO < dfd);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
63 #endif
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
64
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
65 /* Four iterations, with progressively more standard descriptors
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
66 closed. */
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
67 for (i = -1; i <= STDERR_FILENO; i++)
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
68 {
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
69 if (0 <= i)
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
70 ASSERT (close (i) == 0);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
71 dp = opendir (".");
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
72 ASSERT (dp);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
73 ASSERT (dirfd (dp) == -1 || STDERR_FILENO < dirfd (dp));
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
74 ASSERT (closedir (dp) == 0);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
75
13073
03de1efbfdd3 Resolve conflict between the two kinds of module indicators.
Bruno Haible <bruno@clisp.org>
parents: 12559
diff changeset
76 #if HAVE_FDOPENDIR || GNULIB_TEST_FDOPENDIR
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
77 {
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
78 int fd = dup_safer (dfd);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
79 ASSERT (STDERR_FILENO < fd);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
80 dp = fdopendir (fd);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
81 ASSERT (dp);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
82 ASSERT (dirfd (dp) == -1 || STDERR_FILENO < dirfd (dp));
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
83 ASSERT (closedir (dp) == 0);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
84 errno = 0;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
85 ASSERT (close (fd) == -1);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
86 ASSERT (errno == EBADF);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
87 }
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
88 #endif
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
89 }
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
90
13073
03de1efbfdd3 Resolve conflict between the two kinds of module indicators.
Bruno Haible <bruno@clisp.org>
parents: 12559
diff changeset
91 #if HAVE_FDOPENDIR || GNULIB_TEST_FDOPENDIR
11936
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
92 ASSERT (close (dfd) == 0);
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
93 #endif
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
94
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
95 return 0;
a126d5b22410 dirent-safer: new module
Eric Blake <ebb9@byu.net>
parents:
diff changeset
96 }