NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitbe40dd6

committed

Prevent underflow in KeepLogSeg().

The call to XLogGetReplicationSlotMinimumLSN() might return agreater LSN than the one given to the function. Subsequent segmentnumber calculations might then underflow, which could result inunexpected behavior when removing or recyling WAL files. This wasintroduced with max_slot_wal_keep_size inc655077. To fix, skipthe block of code for replication slots if the LSN is greater.Reported-by: Xu XingwangAuthor: Kyotaro HoriguchiReviewed-by: Junwang ZhaoDiscussion:https://postgr.es/m/17903-4288d439dee856c6%40postgresql.orgBackpatch-through: 13

1 parentde2dfa0 commitbe40dd6Copy full SHA for be40dd6

File tree

1 file changed

-1

lines changed

src/backend/access/transam
- xlog.c

1 file changed

-1

lines changed

`‎src/backend/access/transam/xlog.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -9836,7 +9836,7 @@ KeepLogSeg(XLogRecPtr recptr, XLogSegNo *logSegNo)`
`9836`	`9836`	`* max_slot_wal_keep_size.`
`9837`	`9837`	`*/`
`9838`	`9838`	`keep=XLogGetReplicationSlotMinimumLSN();`
`9839`		`-if (keep!=InvalidXLogRecPtr)`
	`9839`	`+if (keep!=InvalidXLogRecPtr&&keep<recptr)`
`9840`	`9840`	`{`
`9841`	`9841`	`XLByteToSeg(keep,segno,wal_segment_size);`
`9842`	`9842`

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!

Commitbe40dd6

File tree

1 file changed

1 file changed

`‎src/backend/access/transam/xlog.c`

0 commit comments