Avoid file descriptor leak in pg_test_fsync.

robertmhaas · robertmhaas · commit 8cf825974cd1 · 2014-11-19T12:14:22.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
@@ -260,8 +260,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
`@@ -260,8 +260,6 @@ test_sync(int writes_per_op)`
`260`	`260`	`}`
`261`	`261`	`else`
`262`	`262`	`{`
`263`		`- if ((tmpfile = open(filename, O_RDWR \| O_DSYNC \| PG_O_DIRECT, 0)) == -1)`
`264`		`- die("could not open output file");`
`265`	`263`	`START_TIMER;`
`266`	`264`	`for (ops = 0; alarm_triggered == false; ops++)`
`267`	`265`	`{`