diff options
| author | Daniel Gustafsson <dgustafsson@postgresql.org> | 2025-02-14 11:50:56 +0100 |
|---|---|---|
| committer | Daniel Gustafsson <dgustafsson@postgresql.org> | 2025-02-14 11:50:56 +0100 |
| commit | cb19cd956f685558b5b6b3d49db6cff01d35726f (patch) | |
| tree | aff08b326bd5557870019533bd1ee8868a97d8c4 | |
| parent | a37c83d1e465399fb1398507b6634f4dce76964c (diff) | |
| download | postgresql-cb19cd956f685558b5b6b3d49db6cff01d35726f.tar.gz postgresql-cb19cd956f685558b5b6b3d49db6cff01d35726f.zip | |
Fix assertion on dereferenced object
Commit 27cc7cd2bc8a accidentally placed the assertion ensuring
that the pointer isn't NULL after it had already been accessed.
Fix by moving the pointer dereferencing to after the assertion.
Backpatch to all supported branches.
Author: Dmitry Koval <d.koval@postgrespro.ru>
Reviewed-by: Daniel Gustafsson <daniel@yesql.se>
Reviewed-by: Michael Paquier <michael@paquier.xyz>
Discussion: https://postgr.es/m/1618848d-cdc7-414b-9c03-08cf4bef4408@postgrespro.ru
Backpatch-through: 13
| -rw-r--r-- | src/backend/executor/execMain.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/backend/executor/execMain.c b/src/backend/executor/execMain.c index 36f34b9edf3..e57b3592981 100644 --- a/src/backend/executor/execMain.c +++ b/src/backend/executor/execMain.c @@ -2631,13 +2631,15 @@ bool EvalPlanQualFetchRowMark(EPQState *epqstate, Index rti, TupleTableSlot *slot) { ExecAuxRowMark *earm = epqstate->relsubs_rowmark[rti - 1]; - ExecRowMark *erm = earm->rowmark; + ExecRowMark *erm; Datum datum; bool isNull; Assert(earm != NULL); Assert(epqstate->origslot != NULL); + erm = earm->rowmark; + if (RowMarkRequiresRowShareLock(erm->markType)) elog(ERROR, "EvalPlanQual doesn't support locking rowmarks"); |