diff options
Diffstat (limited to 'src/backend/jit/llvm/llvmjit_deform.c')
| -rw-r--r-- | src/backend/jit/llvm/llvmjit_deform.c | 24 |
1 files changed, 20 insertions, 4 deletions
diff --git a/src/backend/jit/llvm/llvmjit_deform.c b/src/backend/jit/llvm/llvmjit_deform.c index 835aea83e97..409fc9e8525 100644 --- a/src/backend/jit/llvm/llvmjit_deform.c +++ b/src/backend/jit/llvm/llvmjit_deform.c @@ -22,12 +22,28 @@ #include "access/htup_details.h" #include "access/tupdesc_details.h" +#include "catalog/pg_subscription.h" +#include "catalog/pg_subscription_rel.h" #include "executor/tuptable.h" #include "jit/llvmjit.h" #include "jit/llvmjit_emit.h" /* + * Through an embarrassing oversight, pre-v13 installations may have + * pg_subscription.subslotname and pg_subscription_rel.srsublsn marked as + * attnotnull, which they should not be. To avoid possible crashes, use + * this macro instead of testing attnotnull directly. + */ +#define ATTNOTNULL(att) \ + ((att)->attnotnull && \ + !(((att)->attrelid == SubscriptionRelationId && \ + (att)->attnum == Anum_pg_subscription_subslotname) || \ + ((att)->attrelid == SubscriptionRelRelationId && \ + (att)->attnum == Anum_pg_subscription_rel_srsublsn))) + + +/* * Create a function that deforms a tuple of type desc up to natts columns. */ LLVMValueRef @@ -121,7 +137,7 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, * combination of attisdropped && attnotnull combination shouldn't * exist. */ - if (att->attnotnull && + if (ATTNOTNULL(att) && !att->atthasmissing && !att->attisdropped) guaranteed_column_number = attnum; @@ -419,7 +435,7 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, * into account, because if they're present the heaptuple's natts * would have indicated that a slot_getmissingattrs() is needed. */ - if (!att->attnotnull) + if (!ATTNOTNULL(att)) { LLVMBasicBlockRef b_ifnotnull; LLVMBasicBlockRef b_ifnull; @@ -600,7 +616,7 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, known_alignment = -1; attguaranteedalign = false; } - else if (att->attnotnull && attguaranteedalign && known_alignment >= 0) + else if (ATTNOTNULL(att) && attguaranteedalign && known_alignment >= 0) { /* * If the offset to the column was previously known, a NOT NULL & @@ -610,7 +626,7 @@ slot_compile_deform(LLVMJitContext *context, TupleDesc desc, Assert(att->attlen > 0); known_alignment += att->attlen; } - else if (att->attnotnull && (att->attlen % alignto) == 0) + else if (ATTNOTNULL(att) && (att->attlen % alignto) == 0) { /* * After a NOT NULL fixed-width column with a length that is a |