NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit5407241

committed

pg_waldump: Fix error message for WAL files smaller than XLOG_BLCKSZ.

When opening a WAL file smaller than XLOG_BLCKSZ (e.g. 0 bytes long) whiledetermining the wal_segment_size, pg_waldump checked errno, despite errno notbeing set by the short read. Resulting in a bogus error message.Author: Kyotaro Horiguchi <horikyota.ntt@gmail.com>Discussion:https://postgr.es/m/20220214.181847.775024684568733277.horikyota.ntt@gmail.comBackpatch: 11-, the bug was introducedinfc49e24

1 parent0033fc6 commit5407241Copy full SHA for 5407241

File tree

1 file changed

-8

lines changed

src/bin/pg_waldump
- pg_waldump.c

1 file changed

-8

lines changed

`‎src/bin/pg_waldump/pg_waldump.c`

Lines changed: 5 additions & 8 deletions

Original file line number	Diff line number	Diff line change
`@@ -207,15 +207,12 @@ search_directory(const char directory, const char fname)`
`207`	`207`	`WalSegSz),`
`208`	`208`	`fname,WalSegSz);`
`209`	`209`	`}`
	`210`	`+elseif (r<0)`
	`211`	`+fatal_error("could not read file \"%s\": %s",`
	`212`	`+fname,strerror(errno));`
`210`	`213`	`else`
`211`		`-{`
`212`		`-if (errno!=0)`
`213`		`-fatal_error("could not read file \"%s\": %s",`
`214`		`-fname,strerror(errno));`
`215`		`-else`
`216`		`-fatal_error("could not read file \"%s\": read %d of %zu",`
`217`		`-fname,r, (Size)XLOG_BLCKSZ);`
`218`		`-}`
	`214`	`+fatal_error("could not read file \"%s\": read %d of %zu",`
	`215`	`+fname,r, (Size)XLOG_BLCKSZ);`
`219`	`216`	`close(fd);`
`220`	`217`	`return true;`
`221`	`218`	`}`

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!

Commit5407241

File tree

1 file changed

1 file changed

`‎src/bin/pg_waldump/pg_waldump.c`

0 commit comments