NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit53e1498

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 parent2bc3397 commit53e1498Copy full SHA for 53e1498

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
`@@ -2276,7 +2276,7 @@ pre_sync_fname(char *fname, bool isdir)`
`2276`	`2276`	`{`
`2277`	`2277`	`intfd;`
`2278`	`2278`
`2279`		`-fd=open(fname,O_RDONLY \|PG_BINARY);`
	`2279`	`+fd=BasicOpenFile(fname,O_RDONLY \|PG_BINARY,0);`
`2280`	`2280`
`2281`	`2281`	`/*`
`2282`	`2282`	`* Some OSs don't allow us to open directories at all (Windows returns`
`@@ -2335,7 +2335,7 @@ walkdir(char path, void (action) (char *fname, bool isdir))`
`2335`	`2335`	`#ifndefWIN32`
`2336`	`2336`	`elseif (S_ISLNK(fst.st_mode))`
`2337`	`2337`	`#else`
`2338`		`-elseif (pg_win32_is_junction(subpath))`
	`2338`	`+elseif (pgwin32_is_junction(subpath))`
`2339`	`2339`	`#endif`
`2340`	`2340`	`{`
`2341`	`2341`	`#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!

Commit53e1498

File tree

1 file changed

1 file changed

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

0 commit comments