NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitc1d6ee8

committed

Fix bogus coding in WaitForBackgroundWorkerShutdown().

Some conditions resulted in "return" directly out of a PG_TRY block,which left the exception stack dangling, and to add insult to injuryfailed to restore the state of set_latch_on_sigusr1.This is a bug only in 9.5; in HEAD it was accidentally fixed by commitdb0f6ca, which removed the surrounding PG_TRY block. However, I (tgl)chose to apply the patch to HEAD as well, because the old coding wasgratuitously different from WaitForBackgroundWorkerStartup(), and therewould indeed have been no bug if it were done like that to start with.Dmitry IvanovDiscussion: <1637882.WfYN5gPf1A@abook>

1 parent4c27511 commitc1d6ee8Copy full SHA for c1d6ee8

File tree

1 file changed

-2

lines changed

src/backend/postmaster
- bgworker.c

1 file changed

-2

lines changed

`‎src/backend/postmaster/bgworker.c`

Lines changed: 5 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -1025,13 +1025,16 @@ WaitForBackgroundWorkerShutdown(BackgroundWorkerHandle *handle)`
`1025`	`1025`
`1026`	`1026`	`status=GetBackgroundWorkerPid(handle,&pid);`
`1027`	`1027`	`if (status==BGWH_STOPPED)`
`1028`		`-returnstatus;`
	`1028`	`+break;`
`1029`	`1029`
`1030`	`1030`	`rc=WaitLatch(&MyProc->procLatch,`
`1031`	`1031`	`WL_LATCH_SET \|WL_POSTMASTER_DEATH,0);`
`1032`	`1032`
`1033`	`1033`	`if (rc&WL_POSTMASTER_DEATH)`
`1034`		`-returnBGWH_POSTMASTER_DIED;`
	`1034`	`+{`
	`1035`	`+status=BGWH_POSTMASTER_DIED;`
	`1036`	`+break;`
	`1037`	`+}`
`1035`	`1038`
`1036`	`1039`	`ResetLatch(&MyProc->procLatch);`
`1037`	`1040`	`}`

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!

Commitc1d6ee8

File tree

1 file changed

1 file changed

`‎src/backend/postmaster/bgworker.c`

0 commit comments