From 35e9b1cc1ee296959d52383455052cb3743af478 Mon Sep 17 00:00:00 2001
From: Tom Lane
Date: Fri, 2 Sep 2005 19:02:20 +0000
Subject: Clean up a couple of ad-hoc computations of the maximum number of
 tuples on a page, as suggested by ITAGAKI Takahiro.  Also, change a few
 places that were using some other estimates of max-items-per-page to
 consistently use MaxOffsetNumber.  This is conservatively large --- we could
 have used the new MaxHeapTuplesPerPage macro, or a similar one for index
 tuples --- but those places are simply declaring a fixed-size buffer and
 assuming it will work, rather than actively testing for overrun.  It seems
 safer to size these buffers in a way that can't overflow even if the page is
 corrupt.

---
 src/backend/access/gist/gistvacuum.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

(limited to 'src/backend/access/gist/gistvacuum.c')

diff --git a/src/backend/access/gist/gistvacuum.c b/src/backend/access/gist/gistvacuum.c
index 381cf985590..bcd5ec559c5 100644
--- a/src/backend/access/gist/gistvacuum.c
+++ b/src/backend/access/gist/gistvacuum.c
@@ -8,7 +8,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/access/gist/gistvacuum.c,v 1.6 2005/06/30 17:52:14 teodor Exp $
+ *	  $PostgreSQL: pgsql/src/backend/access/gist/gistvacuum.c,v 1.7 2005/09/02 19:02:19 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -52,7 +52,7 @@ gistVacuumUpdate( GistVacuum *gv, BlockNumber blkno, bool needunion ) {
 	int 		lenaddon=4, curlenaddon=0, ntodelete=0;
 	IndexTuple	idxtuple, *addon=NULL;
 	bool		needwrite=false;
-	OffsetNumber    todelete[ BLCKSZ/SizeOfIptrData ];
+	OffsetNumber    todelete[MaxOffsetNumber];
 	ItemPointerData	*completed=NULL;
 	int 		ncompleted=0, lencompleted=16;
 
@@ -439,7 +439,7 @@ gistbulkdelete(PG_FUNCTION_ARGS) {
 		page   = (Page) BufferGetPage(buffer);
 
 		if ( GistPageIsLeaf(page) ) {
-			OffsetNumber todelete[BLCKSZ/SizeOfIptrData];
+			OffsetNumber todelete[MaxOffsetNumber];
 			int ntodelete = 0;
 
 			LockBuffer(buffer, GIST_UNLOCK);
-- 
cgit v1.2.3