Avoid file descriptor leak in pg_test_fsync.

robertmhaas · robertmhaas · commit a9f3280efee0 · 2014-11-19T12:13:14.000-05:00
This can cause problems on Windows, where files that are still open
can't be unlinked.

Jeff Janes
diff --git a/contrib/pg_test_fsync/pg_test_fsync.c b/contrib/pg_test_fsync/pg_test_fsync.c
@@ -259,8 +259,6 @@ test_sync(int writes_per_op)
 	}
 	else
 	{
-		if ((tmpfile = open(filename, O_RDWR | O_DSYNC | PG_O_DIRECT, 0)) == -1)
-			die("could not open output file");
 		START_TIMER;
 		for (ops = 0; alarm_triggered == false; ops++)
 		{

Original file line number	Diff line number	Diff line change
`@@ -259,8 +259,6 @@ test_sync(int writes_per_op)`
`259`	`259`	`}`
`260`	`260`	`else`
`261`	`261`	`{`
`262`		`- if ((tmpfile = open(filename, O_RDWR \| O_DSYNC \| PG_O_DIRECT, 0)) == -1)`
`263`		`- die("could not open output file");`
`264`	`262`	`START_TIMER;`
`265`	`263`	`for (ops = 0; alarm_triggered == false; ops++)`
`266`	`264`	`{`