pgsql: Change SHA2 implementation based on OpenSSL to use EVP digest ro - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Change SHA2 implementation based on OpenSSL to use EVP digest ro
Date
Msg-id E1kl0HX-0006fX-9v@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Change SHA2 implementation based on OpenSSL to use EVP digest routines

The use of low-level hash routines is not recommended by upstream
OpenSSL since 2000, and pgcrypto already switched to EVP as of 5ff4a67.
This takes advantage of the refactoring done in 87ae969 that has
introduced the allocation and free routines for cryptographic hashes.

Since 1.1.0, OpenSSL does not publish the contents of the cryptohash
contexts, forcing any consumers to rely on OpenSSL for all allocations.
Hence, the resource owner callback mechanism gains a new set of routines
to track and free cryptohash contexts when using OpenSSL, preventing any
risks of leaks in the backend.  Nothing is needed in the frontend thanks
to the refactoring of 87ae969, and the resowner knowledge is isolated
into cryptohash_openssl.c.

Note that this also fixes a failure with SCRAM authentication when using
FIPS in OpenSSL, but as there have been few complaints about this
problem and as this causes an ABI breakage, no backpatch is done.

Author: Michael Paquier
Reviewed-by: Daniel Gustafsson, Heikki Linnakangas
Discussion: https://postgr.es/m/20200924025314.GE7405@paquier.xyz
Discussion: https://postgr.es/m/20180911030250.GA27115@paquier.xyz

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/4f48a6fbe2b28d8281dbbfa2d334fa2ed8472734

Modified Files
--------------
src/backend/replication/basebackup.c  |   8 ++-
src/backend/utils/resowner/resowner.c |  62 ++++++++++++++++
src/common/cryptohash_openssl.c       | 128 +++++++++++++++++++++-------------
src/include/utils/resowner_private.h  |   7 ++
src/tools/pgindent/typedefs.list      |   1 +
5 files changed, 157 insertions(+), 49 deletions(-)


pgsql-committers by date:

Previous
From: Bruce Momjian
Date:
Subject: pgsql: doc: remove unnecessary blank before command option text
Next
From: Michael Paquier
Date:
Subject: pgsql: Rename cryptohashes.c to cryptohashfuncs.c