Commit591b9b0

committed

Use ftruncate() not truncate() in mdunlink. Seems Windows doesn't

support the latter.

1 parentd788c92 commit591b9b0Copy full SHA for 591b9b0

File tree

+18

-2

lines changed

+18

-2

lines changed

Lines changed: 18 additions & 2 deletions

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/storage/smgr/md.c,v 1.131 2007/11/15 21:14:38 momjian Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/storage/smgr/md.c,v 1.132 2007/11/15 21:49:47 tgl Exp $`
`12`	`12`	`*`
`13`	`13`	`*-------------------------------------------------------------------------`
`14`	`14`	`*/`
`@@ -316,7 +316,23 @@ mdunlink(RelFileNode rnode, bool isRedo)`
`316`	`316`	`if (isRedo)`
`317`	`317`	`ret=unlink(path);`
`318`	`318`	`else`
`319`		`-ret=truncate(path,0);`
	`319`	`+{`
	`320`	`+/* truncate(2) would be easier here, but Windows hasn't got it */`
	`321`	`+intfd;`
	`322`	`+`
	`323`	`+fd=BasicOpenFile(path,O_RDWR \|PG_BINARY,0);`
	`324`	`+if (fd >=0)`
	`325`	`+{`
	`326`	`+intsave_errno;`
	`327`	`+`
	`328`	`+ret=ftruncate(fd,0);`
	`329`	`+save_errno=errno;`
	`330`	`+close(fd);`
	`331`	`+errno=save_errno;`
	`332`	`+}`
	`333`	`+else`
	`334`	`+ret=-1;`
	`335`	`+}`
`320`	`336`	`if (ret<0)`
`321`	`337`	`{`
`322`	`338`	`if (!isRedo\|\|errno!=ENOENT)`

Comments

(0)