NotificationsYou must be signed in to change notification settings
Fork0
Star0

Commit231bcd0

committed

Fix incorrect close() call in dsm_impl_mmap().

One improbable error-exit path in this function used close() whereit should have used CloseTransientFile(). This is unlikely to behit in the field, and I think the consequences wouldn't be awful(just an elog(LOG) bleat later). But a bug is a bug, so back-patchto 9.4 where this code came in.Pan BianDiscussion:https://postgr.es/m/152056616579.4966.583293218357089052@wrigleys.postgresql.org

1 parent074251d commit231bcd0Copy full SHA for 231bcd0

File tree

1 file changed

-1

lines changed

src/backend/storage/ipc
- dsm_impl.c

1 file changed

-1

lines changed

`‎src/backend/storage/ipc/dsm_impl.c`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -924,7 +924,7 @@ dsm_impl_mmap(dsm_op op, dsm_handle handle, Size request_size,`
`924`	`924`
`925`	`925`	`/* Back out what's already been done. */`
`926`	`926`	`save_errno=errno;`
`927`		`-close(fd);`
	`927`	`+CloseTransientFile(fd);`
`928`	`928`	`if (op==DSM_OP_CREATE)`
`929`	`929`	`unlink(name);`
`930`	`930`	`errno=save_errno;`

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!

Commit231bcd0

File tree

1 file changed

1 file changed

`‎src/backend/storage/ipc/dsm_impl.c`

0 commit comments