]> granicus.if.org Git - postgresql/commit
Fix incorrect loop counts in tidbitmap.c.
authorTom Lane <tgl@sss.pgh.pa.us>
Fri, 15 Nov 2013 23:34:14 +0000 (18:34 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Fri, 15 Nov 2013 23:34:26 +0000 (18:34 -0500)
commitb18882aed6bf7190677c351199f95d409559bbc0
treee994f84fae21cd68df68422a6bc65fcdfaa9a7f8
parent51b6ae6bba75bca2374a24cf7c740da74c955ad5
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
src/backend/nodes/tidbitmap.c