From d73cc5857faca215ee95c858e836bcc12d1d1b70 Mon Sep 17 00:00:00 2001 From: Bruce Momjian Date: Fri, 21 Mar 2014 13:45:11 -0400 Subject: Properly check for readdir/closedir() failures Clear errno before calling readdir() and handle old MinGW errno bug while adding full test coverage for readdir/closedir failures. Backpatch through 8.4. --- src/port/pgcheckdir.c | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) (limited to 'src/port/pgcheckdir.c') diff --git a/src/port/pgcheckdir.c b/src/port/pgcheckdir.c index 9453bcb68a6..3a2a34cbc61 100644 --- a/src/port/pgcheckdir.c +++ b/src/port/pgcheckdir.c @@ -32,14 +32,12 @@ pg_check_dir(const char *dir) DIR *chkdir; struct dirent *file; - errno = 0; - chkdir = opendir(dir); if (chkdir == NULL) return (errno == ENOENT) ? 0 : -1; - while ((file = readdir(chkdir)) != NULL) + while (errno = 0, (file = readdir(chkdir)) != NULL) { if (strcmp(".", file->d_name) == 0 || strcmp("..", file->d_name) == 0) @@ -55,18 +53,12 @@ pg_check_dir(const char *dir) } #ifdef WIN32 - - /* - * This fix is in mingw cvs (runtime/mingwex/dirent.c rev 1.4), but not in - * released version - */ + /* Bug in old Mingw dirent.c; fixed in mingw-runtime-3.2, 2003-10-10 */ if (GetLastError() == ERROR_NO_MORE_FILES) errno = 0; #endif - closedir(chkdir); - - if (errno != 0) + if (errno || closedir(chkdir)) result = -1; /* some kind of I/O error? */ return result; -- cgit v1.2.3