MAPSIZE macro needs to use MAXALIGN(SizeOfPageHeaderData) instead of

hlinnaka · hlinnaka · commit 7a567d9407de · 2008-12-06T17:31:37.000Z
SizeOfPageHeaderData, like PageGetContents does. Per report by Pavan
Deolasee.
diff --git a/src/backend/access/heap/visibilitymap.c b/src/backend/access/heap/visibilitymap.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/access/heap/visibilitymap.c,v 1.1 2008/12/03 13:05:22 heikki Exp $
+ *	  $PostgreSQL: pgsql/src/backend/access/heap/visibilitymap.c,v 1.2 2008/12/06 17:31:37 heikki Exp $
  *
  * INTERFACE ROUTINES
  *		visibilitymap_clear	- clear a bit in the visibility map
@@ -101,7 +101,7 @@
  * extra headers, so the whole page minus except for the standard page header
  * is used for the bitmap.
  */
-#define MAPSIZE (BLCKSZ - SizeOfPageHeaderData)
+#define MAPSIZE (BLCKSZ - MAXALIGN(SizeOfPageHeaderData))
 
 /* Number of bits allocated for each heap block. */
 #define BITS_PER_HEAPBLOCK 1

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/access/heap/visibilitymap.c,v 1.1 2008/12/03 13:05:22 heikki Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/access/heap/visibilitymap.c,v 1.2 2008/12/06 17:31:37 heikki Exp $`
`12`	`12`	`*`
`13`	`13`	`* INTERFACE ROUTINES`
`14`	`14`	`* visibilitymap_clear - clear a bit in the visibility map`
`@@ -101,7 +101,7 @@`
`101`	`101`	`* extra headers, so the whole page minus except for the standard page header`
`102`	`102`	`* is used for the bitmap.`
`103`	`103`	`*/`
`104`		`-#define MAPSIZE (BLCKSZ - SizeOfPageHeaderData)`
	`104`	`+#define MAPSIZE (BLCKSZ - MAXALIGN(SizeOfPageHeaderData))`
`105`	`105`
`106`	`106`	`/* Number of bits allocated for each heap block. */`
`107`	`107`	`#define BITS_PER_HEAPBLOCK 1`