aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Gaynor <alex.gaynor@gmail.com>2014-03-15 15:42:36 -0700
committerAlex Gaynor <alex.gaynor@gmail.com>2014-03-15 15:42:36 -0700
commit93d7737fe3ef26c9b6b09a951826110e452b939d (patch)
tree3646761a76efa918bece39c9d42b6c62d85a69de
parentc914372fa0a5eb946d843558eb182012555f0d89 (diff)
downloadcryptography-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.py6
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