diff options
author | Alex Gaynor <alex.gaynor@gmail.com> | 2014-03-15 15:42:36 -0700 |
---|---|---|
committer | Alex Gaynor <alex.gaynor@gmail.com> | 2014-03-15 15:42:36 -0700 |
commit | 93d7737fe3ef26c9b6b09a951826110e452b939d (patch) | |
tree | 3646761a76efa918bece39c9d42b6c62d85a69de | |
parent | c914372fa0a5eb946d843558eb182012555f0d89 (diff) | |
download | cryptography-93d7737fe3ef26c9b6b09a951826110e452b939d.tar.gz cryptography-93d7737fe3ef26c9b6b09a951826110e452b939d.tar.bz2 cryptography-93d7737fe3ef26c9b6b09a951826110e452b939d.zip |
Don't put things with side effects in assert statements
-rw-r--r-- | cryptography/hazmat/backends/openssl/backend.py | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/cryptography/hazmat/backends/openssl/backend.py b/cryptography/hazmat/backends/openssl/backend.py index bdbbffd6..251dd09b 100644 --- a/cryptography/hazmat/backends/openssl/backend.py +++ b/cryptography/hazmat/backends/openssl/backend.py @@ -796,7 +796,8 @@ class _RSAVerificationContext(object): # occurs. assert res >= 0 if res == 0: - assert self._backend._consume_errors() + errors = self._backend._consume_errors() + assert errors raise InvalidSignature def _verify_pkcs1(self, rsa_cdata, evp_pkey, evp_md): @@ -813,7 +814,8 @@ class _RSAVerificationContext(object): # occurs. assert res >= 0 if res == 0: - assert self._backend._consume_errors() + errors = self._backend._consume_errors() + assert errors raise InvalidSignature |