]> 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:30 +0000 (18:34 -0500)
commite3a02a39268ff14a8f5f15f1bd9fb59869f26436
tree91542a0cf8465cda70bd60eb5e8a45c85072fcb9
parent71ff6c1ebec774790b4d249abc9aeac79ad0becf
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