diff options
| author |   otto <otto@openbsd.org> | 2006-03-25 07:49:56 +0000 |
|---|---|---|
| committer | otto <otto@openbsd.org> | 2006-03-25 07:49:56 +0000 |
| commit | 075700d30299eea1d1cecc32361b66c5eda47fc5 (patch) | |
| tree | 025443e5d217a613091114f6b568a7f81046c018 | |
| parent | ANSI and KNF. (diff) | |
| download | wireguard-openbsd-075700d30299eea1d1cecc32361b66c5eda47fc5.tar.xz wireguard-openbsd-075700d30299eea1d1cecc32361b66c5eda47fc5.zip | |
Use fstat(3) instead of stat(3) as a matter of principle. We don't
like race conditions, even if the effects of these are void and empty.
| -rw-r--r-- | usr.bin/dc/dc.c | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/usr.bin/dc/dc.c b/usr.bin/dc/dc.c index 6805a23e183..bcaa4abc467 100644 --- a/usr.bin/dc/dc.c +++ b/usr.bin/dc/dc.c @@ -1,4 +1,4 @@ -/* $OpenBSD: dc.c,v 1.8 2006/03/24 21:13:45 otto Exp $ */ +/* $OpenBSD: dc.c,v 1.9 2006/03/25 07:49:56 otto Exp $ */ /* * Copyright (c) 2003, Otto Moerbeek <otto@drijf.net> @@ -17,7 +17,7 @@ */ #ifndef lint -static const char rcsid[] = "$OpenBSD: dc.c,v 1.8 2006/03/24 21:13:45 otto Exp $"; +static const char rcsid[] = "$OpenBSD: dc.c,v 1.9 2006/03/25 07:49:56 otto Exp $"; #endif /* not lint */ #include <sys/stat.h> @@ -89,16 +89,15 @@ main(int argc, char *argv[]) return (0); } if (argc == 1) { - if (stat(argv[0], &st) == -1) + file = fopen(argv[0], "r"); + if (file == NULL) + err(1, "cannot open file %s", argv[0]); + if (fstat(fileno(file), &st) == -1) err(1, "%s", argv[0]); if (S_ISDIR(st.st_mode)) { errno = EISDIR; err(1, "%s", argv[0]); } - - file = fopen(argv[0], "r"); - if (file == NULL) - err(1, "cannot open file %s", argv[0]); src_setstream(&src, file); reset_bmachine(&src); eval(); |