NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitf4060db

committed

Fix inconsequential fd leak in the new mark_file_as_archived() function.

As every error in mark_file_as_archived() will lead to a failure ofpg_basebackup the FD leak couldn't ever lead to a real problem. Itseems better to fix the leak anyway though, rather than silenceCoverity, as the usage of the function might get extended or copied atsome point in the future.Pointed out by Coverity.Backpatch to 9.2, like the relevant part of the previous patch.

1 parentf961ad4 commitf4060dbCopy full SHA for f4060db

File tree

1 file changed

-0

lines changed

src/bin/pg_basebackup
- receivelog.c

1 file changed

-0

lines changed

`‎src/bin/pg_basebackup/receivelog.c`

Lines changed: 3 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -65,6 +65,9 @@ mark_file_as_archived(const char basedir, const char fname)`
`65`	`65`	`{`
`66`	`66`	`fprintf(stderr,_("%s: could not fsync file \"%s\": %s\n"),`
`67`	`67`	`progname,tmppath,strerror(errno));`
	`68`	`+`
	`69`	`+close(fd);`
	`70`	`+`
`68`	`71`	`return false;`
`69`	`72`	`}`
`70`	`73`

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!

Commitf4060db

File tree

1 file changed

1 file changed

`‎src/bin/pg_basebackup/receivelog.c`

0 commit comments