]> 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:21 +0000 (18:34 -0500)
commit4b61f6783793a362423e511d76027e5983fc241c
tree7480c33c0b132c39a72272babaaa8411d06c7bd5
parent1c0dfaa68fa491f2ec4c6f7905246ef07e504d4c
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