diff options
| author | Andrew Gierth <rhodiumtoad@postgresql.org> | 2019-05-23 15:26:01 +0100 |
|---|---|---|
| committer | Andrew Gierth <rhodiumtoad@postgresql.org> | 2019-05-23 15:39:17 +0100 |
| commit | 99efd8d727b0662e152daa2ccd1dcc668ac36056 (patch) | |
| tree | e65b52c315fbdda551672fe56d5d13e56541398d /src/backend/executor/execParallel.c | |
| parent | 2ccebcd23689779b97db1375d038b50bed8dcd68 (diff) | |
| download | postgresql-99efd8d727b0662e152daa2ccd1dcc668ac36056.tar.gz postgresql-99efd8d727b0662e152daa2ccd1dcc668ac36056.zip | |
Fix array size allocation for HashAggregate hash keys.
When there were duplicate columns in the hash key list, the array
sizes could be miscomputed, resulting in access off the end of the
array. Adjust the computation to ensure the array is always large
enough.
(I considered whether the duplicates could be removed in planning, but
I can't rule out the possibility that duplicate columns might have
different hash functions assigned. Simpler to just make sure it works
at execution time regardless.)
Bug apparently introduced in fc4b3dea2 as part of narrowing down the
tuples stored in the hashtable. Reported by Colm McHugh of Salesforce,
though I didn't use their patch. Backpatch back to version 10 where
the bug was introduced.
Discussion: https://postgr.es/m/CAFeeJoKKu0u+A_A9R9316djW-YW3-+Gtgvy3ju655qRHR3jtdA@mail.gmail.com
Diffstat (limited to 'src/backend/executor/execParallel.c')
0 files changed, 0 insertions, 0 deletions