Fix aboriginal bug in BufFileDumpBuffer that would cause it to write the

tglsfdc · tglsfdc · commit 72cbfa4fcc67 · 2007-06-01T23:43:39.000Z
wrong data when dumping a bufferload that crosses a component-file boundary.
This probably has not been seen in the wild because (a) component files are
normally 1GB apiece and (b) non-block-aligned buffer usage is relatively
rare.  But it's fairly easy to reproduce a problem if one reduces RELSEG_SIZE
in a test build.  Kudos to Kurt Harriman for spotting the bug.
diff --git a/src/backend/storage/file/buffile.c b/src/backend/storage/file/buffile.c
@@ -7,7 +7,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/storage/file/buffile.c,v 1.14 2002/09/05 00:43:07 tgl Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/storage/file/buffile.c,v 1.14.2.1 2007/06/01 23:43:39 tgl Exp $
  *
  * NOTES:
  *
@@ -287,7 +287,7 @@ BufFileDumpBuffer(BufFile *file)
 				return;			/* seek failed, give up */
 			file->offsets[file->curFile] = file->curOffset;
 		}
-		bytestowrite = FileWrite(thisfile, file->buffer, bytestowrite);
+		bytestowrite = FileWrite(thisfile, file->buffer + wpos, bytestowrite);
 		if (bytestowrite <= 0)
 			return;				/* failed to write */
 		file->offsets[file->curFile] += bytestowrite;

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`* Portions Copyright (c) 1994, Regents of the University of California`
`8`	`8`	`*`
`9`	`9`	`* IDENTIFICATION`
`10`		`- * $Header: /cvsroot/pgsql/src/backend/storage/file/buffile.c,v 1.14 2002/09/05 00:43:07 tgl Exp $`
	`10`	`+ * $Header: /cvsroot/pgsql/src/backend/storage/file/buffile.c,v 1.14.2.1 2007/06/01 23:43:39 tgl Exp $`
`11`	`11`	`*`
`12`	`12`	`* NOTES:`
`13`	`13`	`*`
`@@ -287,7 +287,7 @@ BufFileDumpBuffer(BufFile *file)`
`287`	`287`	`return; /* seek failed, give up */`
`288`	`288`	`file->offsets[file->curFile] = file->curOffset;`
`289`	`289`	`}`
`290`		`- bytestowrite = FileWrite(thisfile, file->buffer, bytestowrite);`
	`290`	`+ bytestowrite = FileWrite(thisfile, file->buffer + wpos, bytestowrite);`
`291`	`291`	`if (bytestowrite <= 0)`
`292`	`292`	`return; /* failed to write */`
`293`	`293`	`file->offsets[file->curFile] += bytestowrite;`