Add CHECK_FOR_INTERRUPTS while restoring changes during decoding.

Amit Kapila · Amit Kapila · commit 5c51afe23d65 · 2022-10-21T12:08:14.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
@@ -2586,6 +2586,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
`@@ -2586,6 +2586,8 @@ ReorderBufferRestoreChanges(ReorderBuffer rb, ReorderBufferTXN txn,`
`2586`	`2586`	`int readBytes;`
`2587`	`2587`	`ReorderBufferDiskChange *ondisk;`
`2588`	`2588`
	`2589`	`+ CHECK_FOR_INTERRUPTS();`
	`2590`	`+`
`2589`	`2591`	`if (*fd == -1)`
`2590`	`2592`	`{`
`2591`	`2593`	`char path[MAXPGPATH];`