diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2019-06-06 09:46:52 +0300 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2019-06-06 09:46:52 +0300 |
| commit | cd96389d713787b025c84869ba417c6f2f008a86 (patch) | |
| tree | dec68fd47e5d6bf7e94185607d12bf2ce9afc3dd /src/backend/executor/execParallel.c | |
| parent | 6c0c28383749c0ade89ac70ba6dbce9af140c0a9 (diff) | |
| download | postgresql-cd96389d713787b025c84869ba417c6f2f008a86.tar.gz postgresql-cd96389d713787b025c84869ba417c6f2f008a86.zip | |
Fix confusion on different kinds of slots in IndexOnlyScans.
We used the same slot to store a tuple from the index, and to store a
tuple from the table. That's not OK. It worked with the heap, because
heapam_getnextslot() stores a HeapTuple to the slot, and doesn't care how
large the tts_values/nulls arrays are. But when I played with a toy table
AM implementation that used a virtual tuple, it caused memory overruns.
In the passing, tidy up comments on the ioss_PscanLen fields.
Diffstat (limited to 'src/backend/executor/execParallel.c')
0 files changed, 0 insertions, 0 deletions