]> git.ipfire.org Git - thirdparty/postgresql.git/commit
Fix potential memory leak in pgcrypto
authorMichael Paquier <michael@paquier.xyz>
Mon, 19 Oct 2020 00:37:55 +0000 (09:37 +0900)
committerMichael Paquier <michael@paquier.xyz>
Mon, 19 Oct 2020 00:37:55 +0000 (09:37 +0900)
commit57bdf29dd5f66bda9bcb538b6911b1accd89f47e
tree902aee6fc00abfd125615b05d3fa92dfdb8629f0
parent3753e2720a9691d38aa85fe1e641ba55703e2a6f
Fix potential memory leak in pgcrypto

When allocating a EVP context, it would have been possible to leak some
memory allocated directly by OpenSSL, that PostgreSQL lost track of if
the initialization of the context allocated failed.  The cleanup can be
done with EVP_MD_CTX_destroy().

Note that EVP APIs exist since OpenSSL 0.9.7 and we have in the tree
equivalent implementations for older versions since ce9b75d (code
removed with 9b7cd59a as of 10~).  However, in 9.5 and 9.6, the existing
code makes use of EVP_MD_CTX_destroy() and EVP_MD_CTX_create() without
an equivalent implementation when building the tree with OpenSSL 0.9.6
or older, meaning that this code is in reality broken with such versions
since it got introduced in e2838c5.  As we have heard no complains about
that, it does not seem worth bothering with in 9.5 and 9.6, so I have
left that out for simplicity.

Author: Michael Paquier
Discussion: https://postgr.es/m/20201015072212.GC2305@paquier.xyz
Backpatch-through: 9.5
contrib/pgcrypto/openssl.c