NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit6fd6669

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 parent14de825 commit6fd6669Copy full SHA for 6fd6669

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
`@@ -2456,7 +2456,7 @@ pre_sync_fname(char *fname, bool isdir)`
`2456`	`2456`	`{`
`2457`	`2457`	`intfd;`
`2458`	`2458`
`2459`		`-fd=open(fname,O_RDONLY \|PG_BINARY);`
	`2459`	`+fd=BasicOpenFile(fname,O_RDONLY \|PG_BINARY,0);`
`2460`	`2460`
`2461`	`2461`	`/*`
`2462`	`2462`	`* Some OSs don't allow us to open directories at all (Windows returns`
`@@ -2515,7 +2515,7 @@ walkdir(char path, void (action) (char *fname, bool isdir))`
`2515`	`2515`	`#ifndefWIN32`
`2516`	`2516`	`elseif (S_ISLNK(fst.st_mode))`
`2517`	`2517`	`#else`
`2518`		`-elseif (pg_win32_is_junction(subpath))`
	`2518`	`+elseif (pgwin32_is_junction(subpath))`
`2519`	`2519`	`#endif`
`2520`	`2520`	`{`
`2521`	`2521`	`#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!

Commit6fd6669

File tree

1 file changed

1 file changed

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

0 commit comments