NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commitab69ea9

committed

Fix assertion failure for SSL connections.

Commitcfdf4dc added an assertion that every WaitLatch() or similarhandles postmaster death. One place did not, but was missed inreview and testing due to the need for an SSL connection. Fix, byasking for WL_EXIT_ON_PM_DEATH.Reported-by: Christoph BergDiscussion:https://postgr.es/m/20181124143845.GA15039%40msg.df7cb.de

1 parentd5890f4 commitab69ea9Copy full SHA for ab69ea9

File tree

1 file changed

-2

lines changed

src/backend/libpq
- be-secure-openssl.c

1 file changed

-2

lines changed

`‎src/backend/libpq/be-secure-openssl.c`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -406,9 +406,9 @@ be_tls_open_server(Port *port)`
`406`	`406`	`* StartupPacketTimeoutHandler() which directly exits.`
`407`	`407`	`*/`
`408`	`408`	`if (err==SSL_ERROR_WANT_READ)`
`409`		`-waitfor=WL_SOCKET_READABLE;`
	`409`	`+waitfor=WL_SOCKET_READABLE \|WL_EXIT_ON_PM_DEATH;`
`410`	`410`	`else`
`411`		`-waitfor=WL_SOCKET_WRITEABLE;`
	`411`	`+waitfor=WL_SOCKET_WRITEABLE \|WL_EXIT_ON_PM_DEATH;`
`412`	`412`
`413`	`413`	`(void)WaitLatchOrSocket(MyLatch,waitfor,port->sock,0,`
`414`	`414`	`WAIT_EVENT_SSL_OPEN_SERVER);`

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!

Commitab69ea9

File tree

1 file changed

1 file changed

`‎src/backend/libpq/be-secure-openssl.c`

0 commit comments