diff options
| author | Amit Kapila <akapila@postgresql.org> | 2022-08-23 09:10:28 +0530 |
|---|---|---|
| committer | Amit Kapila <akapila@postgresql.org> | 2022-08-23 09:10:28 +0530 |
| commit | 4985a45917c88ae286ed76971154668e7d545832 (patch) | |
| tree | 4ab423367442ddb04e6c880f1bf50faa83443818 | |
| parent | 9f0073ef7dac2837c45e7960d1157d4a69046b3c (diff) | |
| download | postgresql-4985a45917c88ae286ed76971154668e7d545832.tar.gz postgresql-4985a45917c88ae286ed76971154668e7d545832.zip | |
Add CHECK_FOR_INTERRUPTS while decoding changes.
While decoding changes in a loop, if we skip all the changes there is no
CFI making the loop uninterruptible.
Reported-by: Whale Song and Andrey Borodin
Bug: 17580
Author: Masahiko Sawada
Reviwed-by: Amit Kapila
Backpatch-through: 10
Discussion: https://postgr.es/m/17580-849c1d5b6d7eb422@postgresql.org
Discussion: https://postgr.es/m/B319ECD6-9A28-4CDF-A8F4-3591E0BF2369@yandex-team.ru
| -rw-r--r-- | src/backend/replication/logical/reorderbuffer.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/src/backend/replication/logical/reorderbuffer.c b/src/backend/replication/logical/reorderbuffer.c index ef8c2ea6dfe..368f60159da 100644 --- a/src/backend/replication/logical/reorderbuffer.c +++ b/src/backend/replication/logical/reorderbuffer.c @@ -1577,6 +1577,8 @@ ReorderBufferCommit(ReorderBuffer *rb, TransactionId xid, Relation relation = NULL; Oid reloid; + CHECK_FOR_INTERRUPTS(); + switch (change->action) { case REORDER_BUFFER_CHANGE_INTERNAL_SPEC_CONFIRM: |