diff options
author | Alex Gaynor <alex.gaynor@gmail.com> | 2015-03-24 19:33:43 -0400 |
---|---|---|
committer | Alex Gaynor <alex.gaynor@gmail.com> | 2015-03-24 19:33:43 -0400 |
commit | fbd7d7f2adb5dfb66175e5a2f77e4e0c4a640107 (patch) | |
tree | a1e8d222f91b122c33b172501639a0f7c269457a /src | |
parent | 1a12b51dda7029cbff9ffd15fb0c258a4de9e6aa (diff) | |
download | cryptography-fbd7d7f2adb5dfb66175e5a2f77e4e0c4a640107.tar.gz cryptography-fbd7d7f2adb5dfb66175e5a2f77e4e0c4a640107.tar.bz2 cryptography-fbd7d7f2adb5dfb66175e5a2f77e4e0c4a640107.zip |
Fixes #1776 -- ensure that we only init bindings once per process
Diffstat (limited to 'src')
-rw-r--r-- | src/cryptography/hazmat/bindings/commoncrypto/binding.py | 15 | ||||
-rw-r--r-- | src/cryptography/hazmat/bindings/openssl/binding.py | 17 |
2 files changed, 20 insertions, 12 deletions
diff --git a/src/cryptography/hazmat/bindings/commoncrypto/binding.py b/src/cryptography/hazmat/bindings/commoncrypto/binding.py index f48b59cb..0a284ab6 100644 --- a/src/cryptography/hazmat/bindings/commoncrypto/binding.py +++ b/src/cryptography/hazmat/bindings/commoncrypto/binding.py @@ -4,6 +4,8 @@ from __future__ import absolute_import, division, print_function +import threading + from cryptography.hazmat.bindings.utils import ( build_ffi_for_binding, load_library_for_binding, ) @@ -36,6 +38,7 @@ class Binding(object): ], ) lib = None + _init_lock = threading.Lock() def __init__(self): self._ensure_ffi_initialized() @@ -45,8 +48,10 @@ class Binding(object): if cls.lib is not None: return - cls.lib = load_library_for_binding( - cls.ffi, - module_prefix=cls._module_prefix, - modules=cls._modules, - ) + with self._init_lock: + if cls.lib is None: + cls.lib = load_library_for_binding( + cls.ffi, + module_prefix=cls._module_prefix, + modules=cls._modules, + ) diff --git a/src/cryptography/hazmat/bindings/openssl/binding.py b/src/cryptography/hazmat/bindings/openssl/binding.py index cf456e9f..7f379f08 100644 --- a/src/cryptography/hazmat/bindings/openssl/binding.py +++ b/src/cryptography/hazmat/bindings/openssl/binding.py @@ -98,6 +98,7 @@ class Binding(object): _locks = None _lock_cb_handle = None + _init_lock = threading.Lock() _lock_init_lock = threading.Lock() ffi = build_ffi_for_binding( @@ -117,14 +118,16 @@ class Binding(object): if cls.lib is not None: return - cls.lib = load_library_for_binding( - cls.ffi, - cls._module_prefix, - cls._modules, - ) + with self._init_lock: + if cls.lib is None: + cls.lib = load_library_for_binding( + cls.ffi, + cls._module_prefix, + cls._modules, + ) - res = cls.lib.Cryptography_add_osrandom_engine() - assert res != 0 + res = cls.lib.Cryptography_add_osrandom_engine() + assert res != 0 @classmethod def init_static_locks(cls): |