Add CHECK_FOR_INTERRUPTS while restoring changes during decoding.

Amit Kapila · Amit Kapila · commit 10eaa975018f · 2022-10-21T12:43:28.000+05:30
Previously in commit 42681df, we added CFI during decoding changes but missed another similar case that can happen while restoring changes spilled to disk back into memory in a loop. Reported-by: Robert Haas Author: Amit Kapila Backpatch-through: 10 Discussion: https://postgr.es/m/CA+TgmoaLObg0QbstbC8ykDwOdD1bDkr4AbPpB=0DPgA2JW0mFg@mail.gmail.com
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c
@@ -4110,6 +4110,8 @@ ReorderBufferRestoreChanges(ReorderBuffer *rb, ReorderBufferTXN *txn,
 		int			readBytes;
 		ReorderBufferDiskChange *ondisk;
 
+		CHECK_FOR_INTERRUPTS();
+
 		if (*fd == -1)
 		{
 			char		path[MAXPGPATH];

Original file line number	Diff line number	Diff line change
`@@ -4110,6 +4110,8 @@ ReorderBufferRestoreChanges(ReorderBuffer rb, ReorderBufferTXN txn,`
`4110`	`4110`	`int readBytes;`
`4111`	`4111`	`ReorderBufferDiskChange *ondisk;`
`4112`	`4112`
	`4113`	`+ CHECK_FOR_INTERRUPTS();`
	`4114`	`+`
`4113`	`4115`	`if (*fd == -1)`
`4114`	`4116`	`{`
`4115`	`4117`	`char path[MAXPGPATH];`