]> granicus.if.org Git - postgresql/commit
Remove duplicate setting of SSL_OP_SINGLE_DH_USE option.
authorTom Lane <tgl@sss.pgh.pa.us>
Wed, 2 Aug 2017 15:28:46 +0000 (11:28 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Wed, 2 Aug 2017 15:28:49 +0000 (11:28 -0400)
commitf352f91cbf2f662c4f043d3650010b02da0cde1c
treefad7bcfd5086d41363bae00ad6ed0367014e7429
parent41cefbb6db58c574e086efef2773a978f108d717
Remove duplicate setting of SSL_OP_SINGLE_DH_USE option.

Commit c0a15e07c moved the setting of OpenSSL's SSL_OP_SINGLE_DH_USE option
into a new subroutine initialize_dh(), but forgot to remove it from where
it was.  SSL_CTX_set_options() is a trivial function, amounting indeed to
just "ctx->options |= op", hence there's no reason to contort the code or
break separation of concerns to avoid calling it twice.  So separating the
DH setup from disabling of old protocol versions is a good change, but we
need to finish the job.

Noted while poking into the question of SSL session tickets.
src/backend/libpq/be-secure-openssl.c