NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commite7bab08

committed

Further tweaking of the readfile() function in pg_ctl.

Don't leak a file descriptor if the file is empty or we can't read its size.Expect there to be a newline at the end of the last line, too. If thereisn't, ignore anything after the last newline. This makes it a tiny bitmore robust in case the file is appended to concurrently, so that we don'treturn the last line if it hasn't been fully written yet. And this makesthe code a bit less obscure, anyway. Per Tom Lane's suggestion.Backpatch to all supported branches.

1 parent623e49c commite7bab08Copy full SHA for e7bab08

File tree

1 file changed

+11

-4

lines changed

src/bin/pg_ctl
- pg_ctl.c

1 file changed

+11

-4

lines changed

`‎src/bin/pg_ctl/pg_ctl.c‎`

Lines changed: 11 additions & 4 deletions

Original file line number	Diff line number	Diff line change
`@@ -333,10 +333,14 @@ readfile(const char *path)`
`333`	`333`	`if (fd<0)`
`334`	`334`	`returnNULL;`
`335`	`335`	`if (fstat(fd,&statbuf)<0)`
	`336`	`+{`
	`337`	`+close(fd);`
`336`	`338`	`returnNULL;`
	`339`	`+}`
`337`	`340`	`if (statbuf.st_size==0)`
`338`	`341`	`{`
`339`	`342`	`/* empty file */`
	`343`	`+close(fd);`
`340`	`344`	`result= (char*)pg_malloc(sizeof(char));`
`341`	`345`	`*result=NULL;`
`342`	`346`	`returnresult;`
`@@ -352,14 +356,17 @@ readfile(const char *path)`
`352`	`356`	`returnNULL;`
`353`	`357`	`}`
`354`	`358`
`355`		`-/* count newlines */`
	`359`	`+/*`
	`360`	`+ * Count newlines. We expect there to be a newline after each full line,`
	`361`	`+ * including one at the end of file. If there isn't a newline at the end,`
	`362`	`+ * any characters after the last newline will be ignored.`
	`363`	`+ */`
`356`	`364`	`nlines=0;`
`357`		`-for (i=0;i<len-1;i++)`
	`365`	`+for (i=0;i<len;i++)`
`358`	`366`	`{`
`359`	`367`	`if (buffer[i]=='\n')`
`360`	`368`	`nlines++;`
`361`	`369`	`}`
`362`		`-nlines++;/* account for the last line */`
`363`	`370`
`364`	`371`	`/* set up the result buffer */`
`365`	`372`	`result= (char*)pg_malloc((nlines+1)sizeof(char*));`
`@@ -369,7 +376,7 @@ readfile(const char *path)`
`369`	`376`	`n=0;`
`370`	`377`	`for (i=0;i<len;i++)`
`371`	`378`	`{`
`372`		`-if (buffer[i]=='\n'\|\|i==len-1)`
	`379`	`+if (buffer[i]=='\n')`
`373`	`380`	`{`
`374`	`381`	`intslen=&buffer[i]-linebegin+1;`
`375`	`382`	`char*linebuf=pg_malloc(slen+1);`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commite7bab08

File tree

1 file changed

1 file changed

`‎src/bin/pg_ctl/pg_ctl.c‎`

0 commit comments