NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit8046465

committed

Fix pg_basebackup output to stdout on Windows.

When writing a backup to stdout with pg_basebackup on Windows, put stdoutto binary mode. Any CR bytes in the output will otherwise be outputincorrectly as CR+LF.In the passing, standardize on using "_setmode" instead of "setmode", forthe sake of consistency. They both do the same thing, but according toMSDN documentation, setmode is deprecated.Fixes bug #14634, reported by Henry Boehlert. Patch by Haribabu Kommi.Backpatch to all supported versions.Discussion:https://www.postgresql.org/message-id/20170428082818.24366.13134@wrigleys.postgresql.org

1 parenta3ca72a commit8046465Copy full SHA for 8046465

File tree

2 files changed

-2

lines changed

src/bin
- pg_basebackup
  - pg_basebackup.c
- pg_dump
  - pg_backup_archiver.c

2 files changed

-2

lines changed

`‎src/bin/pg_basebackup/pg_basebackup.c`

Lines changed: 4 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -954,6 +954,10 @@ ReceiveTarFile(PGconn conn, PGresult res, int rownum)`
`954`	`954`	`*/`
`955`	`955`	`if (strcmp(basedir,"-")==0)`
`956`	`956`	`{`
	`957`	`+#ifdefWIN32`
	`958`	`+_setmode(fileno(stdout),_O_BINARY);`
	`959`	`+#endif`
	`960`	`+`
`957`	`961`	`#ifdefHAVE_LIBZ`
`958`	`962`	`if (compresslevel!=0)`
`959`	`963`	`{`

`‎src/bin/pg_dump/pg_backup_archiver.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -2346,9 +2346,9 @@ _allocAH(const char *FileSpec, const ArchiveFormat fmt,`
`2346`	`2346`	`(AH->fSpec==NULL\|\|strcmp(AH->fSpec,"")==0))`
`2347`	`2347`	`{`
`2348`	`2348`	`if (mode==archModeWrite)`
`2349`		`-setmode(fileno(stdout),O_BINARY);`
	`2349`	`+_setmode(fileno(stdout),O_BINARY);`
`2350`	`2350`	`else`
`2351`		`-setmode(fileno(stdin),O_BINARY);`
	`2351`	`+_setmode(fileno(stdin),O_BINARY);`
`2352`	`2352`	`}`
`2353`	`2353`	`#endif`
`2354`	`2354`

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!

Commit8046465

File tree

2 files changed

2 files changed

`‎src/bin/pg_basebackup/pg_basebackup.c`

`‎src/bin/pg_dump/pg_backup_archiver.c`

0 commit comments