diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-21 21:18:30 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2011-02-21 21:18:30 -0500 |
| commit | 1487ca0514fe18837f8c2c3f4fcfc37e465c0811 (patch) | |
| tree | 6339ebe7f0e26285a43b61aa75ea263ec2e702db /src/include | |
| parent | 69bef7ff79eb33e8a355e79e7fbb5338d1557775 (diff) | |
| download | postgresql-1487ca0514fe18837f8c2c3f4fcfc37e465c0811.tar.gz postgresql-1487ca0514fe18837f8c2c3f4fcfc37e465c0811.zip | |
Fix dangling-pointer problem in before-row update trigger processing.
ExecUpdate checked for whether ExecBRUpdateTriggers had returned a new
tuple value by seeing if the returned tuple was pointer-equal to the old
one. But the "old one" was in estate->es_junkFilter's result slot, which
would be scribbled on if we had done an EvalPlanQual update in response to
a concurrent update of the target tuple; therefore we were comparing a
dangling pointer to a live one. Given the right set of circumstances we
could get a false match, resulting in not forcing the tuple to be stored in
the slot we thought it was stored in. In the case reported by Maxim Boguk
in bug #5798, this led to "cannot extract system attribute from virtual
tuple" failures when trying to do "RETURNING ctid". I believe there is a
very-low-probability chance of more serious errors, such as generating
incorrect index entries based on the original rather than the
trigger-modified version of the row.
In HEAD, change all of ExecBRInsertTriggers, ExecIRInsertTriggers,
ExecBRUpdateTriggers, and ExecIRUpdateTriggers so that they continue to
have similar APIs. In the back branches I just changed
ExecBRUpdateTriggers, since there is no bug in the ExecBRInsertTriggers
case.
Diffstat (limited to 'src/include')
| -rw-r--r-- | src/include/commands/trigger.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/include/commands/trigger.h b/src/include/commands/trigger.h index d88f5848854..12f494726f5 100644 --- a/src/include/commands/trigger.h +++ b/src/include/commands/trigger.h @@ -147,10 +147,10 @@ extern void ExecBSUpdateTriggers(EState *estate, ResultRelInfo *relinfo); extern void ExecASUpdateTriggers(EState *estate, ResultRelInfo *relinfo); -extern HeapTuple ExecBRUpdateTriggers(EState *estate, +extern TupleTableSlot *ExecBRUpdateTriggers(EState *estate, ResultRelInfo *relinfo, ItemPointer tupleid, - HeapTuple newtuple, + TupleTableSlot *slot, CommandId cid); extern void ExecARUpdateTriggers(EState *estate, ResultRelInfo *relinfo, |