NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitcd83cb9

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 parent73c61a5 commitcd83cb9Copy full SHA for cd83cb9

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
`@@ -222,15 +222,12 @@ search_directory(const char directory, const char fname)`
`222`	`222`	`WalSegSz),`
`223`	`223`	`fname,WalSegSz);`
`224`	`224`	`}`
	`225`	`+elseif (r<0)`
	`226`	`+fatal_error("could not read file \"%s\": %m",`
	`227`	`+fname);`
`225`	`228`	`else`
`226`		`-{`
`227`		`-if (errno!=0)`
`228`		`-fatal_error("could not read file \"%s\": %m",`
`229`		`-fname);`
`230`		`-else`
`231`		`-fatal_error("could not read file \"%s\": read %d of %d",`
`232`		`-fname,r,XLOG_BLCKSZ);`
`233`		`-}`
	`229`	`+fatal_error("could not read file \"%s\": read %d of %d",`
	`230`	`+fname,r,XLOG_BLCKSZ);`
`234`	`231`	`close(fd);`
`235`	`232`	`return true;`
`236`	`233`	`}`

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!

Commitcd83cb9

File tree

1 file changed

1 file changed

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

0 commit comments