diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2013-10-07 23:57:40 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2013-10-08 00:14:15 +0300 |
| commit | 42c63cafb2af32795afa7dd5d3690186af313785 (patch) | |
| tree | dddcd8622cceca1b2ea4ff5377987b57fd7dce60 /src/backend/access/heap/heapam.c | |
| parent | 1c4dfd19a6ebb3c2a0471f0a57b9f13d0ccd9d02 (diff) | |
Fix bugs in SSI tuple locking.
1. In heap_hot_search_buffer(), the PredicateLockTuple() call is passed
wrong offset number. heapTuple->t_self is set to the tid of the first
tuple in the chain that's visited, not the one actually being read.
2. CheckForSerializableConflictIn() uses the tuple's t_ctid field
instead of t_self to check for exiting predicate locks on the tuple. If
the tuple was updated, but the updater rolled back, t_ctid points to the
aborted dead tuple.
Reported by Hannu Krosing. Backpatch to 9.1.
Diffstat (limited to 'src/backend/access/heap/heapam.c')
| -rw-r--r-- | src/backend/access/heap/heapam.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/backend/access/heap/heapam.c b/src/backend/access/heap/heapam.c index 5c1edafdfd0..9e8de15d240 100644 --- a/src/backend/access/heap/heapam.c +++ b/src/backend/access/heap/heapam.c @@ -1555,6 +1555,8 @@ heap_hot_search_buffer(ItemPointer tid, Relation relation, Buffer buffer, offnum = ItemPointerGetOffsetNumber(tid); at_chain_start = true; + heapTuple->t_self = *tid; + /* Scan through possible multiple members of HOT-chain */ for (;;) { @@ -1586,6 +1588,7 @@ heap_hot_search_buffer(ItemPointer tid, Relation relation, Buffer buffer, heapTuple.t_len = ItemIdGetLength(lp); heapTuple.t_tableOid = relation->rd_id; heapTuple.t_self = *tid; + ItemPointerSetOffsetNumber(&heapTuple.t_self, offnum); /* * Shouldn't see a HEAP_ONLY tuple at chain start. |