Add CHECK_FOR_INTERRUPTS while restoring changes during decoding.

Amit Kapila · Amit Kapila · commit 61838d2dd3c4 · 2022-10-21T11:54:34.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
@@ -2534,6 +2534,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
`@@ -2534,6 +2534,8 @@ ReorderBufferRestoreChanges(ReorderBuffer rb, ReorderBufferTXN txn,`
`2534`	`2534`	`int readBytes;`
`2535`	`2535`	`ReorderBufferDiskChange *ondisk;`
`2536`	`2536`
	`2537`	`+ CHECK_FOR_INTERRUPTS();`
	`2538`	`+`
`2537`	`2539`	`if (*fd == -1)`
`2538`	`2540`	`{`
`2539`	`2541`	`char path[MAXPGPATH];`