mirror of
https://github.com/postgres/postgres.git
synced 2025-04-24 10:47:04 +03:00
pg_upgrade: properly detect file copy failure on Windows
Previously, file copy failures were ignored on Windows due to an incorrect return value check. Report by Manu Joye Backpatch through 9.1
This commit is contained in:
parent
42aa1c032e
commit
025106e314
@ -34,7 +34,11 @@ copyAndUpdateFile(pageCnvCtx *pageConverter,
|
||||
{
|
||||
if (pageConverter == NULL)
|
||||
{
|
||||
if (pg_copy_file(src, dst, force) == -1)
|
||||
#ifndef WIN32
|
||||
if (copy_file(src, dst, force) == -1)
|
||||
#else
|
||||
if (CopyFile(src, dst, force) == 0)
|
||||
#endif
|
||||
return getErrorText(errno);
|
||||
else
|
||||
return NULL;
|
||||
|
@ -70,7 +70,6 @@ extern char *output_files[];
|
||||
|
||||
|
||||
#ifndef WIN32
|
||||
#define pg_copy_file copy_file
|
||||
#define pg_mv_file rename
|
||||
#define pg_link_file link
|
||||
#define PATH_SEPARATOR '/'
|
||||
@ -82,7 +81,6 @@ extern char *output_files[];
|
||||
#define ECHO_QUOTE "'"
|
||||
#define ECHO_BLANK ""
|
||||
#else
|
||||
#define pg_copy_file CopyFile
|
||||
#define pg_mv_file pgrename
|
||||
#define pg_link_file win32_pghardlink
|
||||
#define PATH_SEPARATOR '\\'
|
||||
|
Loading…
x
Reference in New Issue
Block a user