diff options
author | Gregory P. Smith <greg@krypto.org> | 2013-02-02 01:05:29 (GMT) |
---|---|---|
committer | Gregory P. Smith <greg@krypto.org> | 2013-02-02 01:05:29 (GMT) |
commit | aded2e5e59a122962ccbf8058cd306da344f27b9 (patch) | |
tree | 88d25a2f4eacb7bf1e5946714e0b49a8632be567 /Modules/_hashopenssl.c | |
parent | ce9e3c3af9624fa5273b2e301043f3892135fb18 (diff) | |
download | cpython-aded2e5e59a122962ccbf8058cd306da344f27b9.zip cpython-aded2e5e59a122962ccbf8058cd306da344f27b9.tar.gz cpython-aded2e5e59a122962ccbf8058cd306da344f27b9.tar.bz2 |
In the _hashlib module, only initialize the static data for OpenSSL's
constructors once, to avoid memory leaks when finalizing and re-initializing
the Python interpreter.
Diffstat (limited to 'Modules/_hashopenssl.c')
-rw-r--r-- | Modules/_hashopenssl.c | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/Modules/_hashopenssl.c b/Modules/_hashopenssl.c index dd4317f..8bcf71d 100644 --- a/Modules/_hashopenssl.c +++ b/Modules/_hashopenssl.c @@ -70,7 +70,7 @@ static PyTypeObject EVPtype; #define DEFINE_CONSTS_FOR_NEW(Name) \ - static PyObject *CONST_ ## Name ## _name_obj; \ + static PyObject *CONST_ ## Name ## _name_obj = NULL; \ static EVP_MD_CTX CONST_new_ ## Name ## _ctx; \ static EVP_MD_CTX *CONST_new_ ## Name ## _ctx_p = NULL; @@ -587,12 +587,15 @@ generate_hash_name_list(void) " hash object; optionally initialized with a string") \ } -/* used in the init function to setup a constructor */ +/* used in the init function to setup a constructor: initialize OpenSSL + constructor constants if they haven't been initialized already. */ #define INIT_CONSTRUCTOR_CONSTANTS(NAME) do { \ - CONST_ ## NAME ## _name_obj = PyUnicode_FromString(#NAME); \ - if (EVP_get_digestbyname(#NAME)) { \ - CONST_new_ ## NAME ## _ctx_p = &CONST_new_ ## NAME ## _ctx; \ - EVP_DigestInit(CONST_new_ ## NAME ## _ctx_p, EVP_get_digestbyname(#NAME)); \ + if (CONST_ ## NAME ## _name_obj == NULL) { \ + CONST_ ## NAME ## _name_obj = PyUnicode_FromString(#NAME); \ + if (EVP_get_digestbyname(#NAME)) { \ + CONST_new_ ## NAME ## _ctx_p = &CONST_new_ ## NAME ## _ctx; \ + EVP_DigestInit(CONST_new_ ## NAME ## _ctx_p, EVP_get_digestbyname(#NAME)); \ + } \ } \ } while (0); |