diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2013-11-15 18:34:14 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2013-11-15 18:34:14 -0500 |
| commit | f1f21b2d6fd170faf9824306ef4f4950c32ce49d (patch) | |
| tree | 0ba3d0d95fd47cbb20936c1e82164b1f4fb6b2c6 /src/backend/commands/aggregatecmds.c | |
| parent | 97e1ec467099f1e581f491c8a57b7d56d0b9c539 (diff) | |
| download | postgresql-f1f21b2d6fd170faf9824306ef4f4950c32ce49d.tar.gz postgresql-f1f21b2d6fd170faf9824306ef4f4950c32ce49d.zip | |
Fix incorrect loop counts in tidbitmap.c.
A couple of places that should have been iterating over WORDS_PER_CHUNK
words were iterating over WORDS_PER_PAGE words instead. This thinko
accidentally failed to fail, because (at least on common architectures
with default BLCKSZ) WORDS_PER_CHUNK is a bit less than WORDS_PER_PAGE,
and the extra words being looked at were always zero so nothing happened.
Still, it's a bug waiting to happen if anybody ever fools with the
parameters affecting TIDBitmap sizes, and it's a small waste of cycles
too. So back-patch to all active branches.
Etsuro Fujita
Diffstat (limited to 'src/backend/commands/aggregatecmds.c')
0 files changed, 0 insertions, 0 deletions