NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit262fbcb

committed

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.

1 parent30ba0d0 commit262fbcbCopy full SHA for 262fbcb

File tree

1 file changed

-2

lines changed

src/backend/storage/file
- fd.c

1 file changed

-2

lines changed

`‎src/backend/storage/file/fd.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2015,7 +2015,7 @@ pre_sync_fname(char *fname, bool isdir)`
`2015`	`2015`	`{`
`2016`	`2016`	`intfd;`
`2017`	`2017`
`2018`		`-fd=open(fname,O_RDONLY \|PG_BINARY);`
	`2018`	`+fd=BasicOpenFile(fname,O_RDONLY \|PG_BINARY,0);`
`2019`	`2019`
`2020`	`2020`	`/*`
`2021`	`2021`	`* Some OSs don't allow us to open directories at all (Windows returns`
`@@ -2074,7 +2074,7 @@ walkdir(char path, void (action) (char *fname, bool isdir))`
`2074`	`2074`	`#ifndefWIN32`
`2075`	`2075`	`elseif (S_ISLNK(fst.st_mode))`
`2076`	`2076`	`#else`
`2077`		`-elseif (pg_win32_is_junction(subpath))`
	`2077`	`+elseif (pgwin32_is_junction(subpath))`
`2078`	`2078`	`#endif`
`2079`	`2079`	`{`
`2080`	`2080`	`#if defined(HAVE_READLINK)\|\| defined(WIN32)`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit262fbcb

File tree

1 file changed

1 file changed

`‎src/backend/storage/file/fd.c`

0 commit comments