postgres.git / summary / log / commit / refs

commit    603fe0181ad083ddc83f291fdce8ddc43d617154
Author:   Robert Haas <rhaas@postgresql.org>
Date:     Tue May 05 12:30:28 2015 +0000
First released in: 9.5.0, 9.0.20, 9.1.16, 9.2.11, 9.3.7, 9.4.2

    Fix some problems with patch to fsync the data directory.
    
    pg_win32_is_junction() was a typo for pgwin32_is_junction().  open()
    was used not only in a two-argument form, which breaks on Windows,
    but also where BasicOpenFile() should have been used.
    
    Per reports from Andrew Dunstan and David Rowley.


 src/backend/storage/file/fd.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/backend/storage/file/fd.c b/src/backend/storage/file/fd.c
index f34e618ac5a..295eeb040ae 100644
--- a/src/backend/storage/file/fd.c
+++ b/src/backend/storage/file/fd.c
@@ -2449,7 +2449,7 @@ pre_sync_fname(char *fname, bool isdir)
 {
 	int			fd;
 
-	fd = open(fname, O_RDONLY | PG_BINARY);
+	fd = BasicOpenFile(fname, O_RDONLY | PG_BINARY, 0);
 
 	/*
 	 * Some OSs don't allow us to open directories at all (Windows returns
@@ -2508,7 +2508,7 @@ walkdir(char *path, void (*action) (char *fname, bool isdir))
 #ifndef WIN32
 		else if (S_ISLNK(fst.st_mode))
 #else
-		else if (pg_win32_is_junction(subpath))
+		else if (pgwin32_is_junction(subpath))
 #endif
 		{
 #if defined(HAVE_READLINK) || defined(WIN32)


[parent: d8ac77ab178d]