diff options
| author | Paul Kehrer <paul.l.kehrer@gmail.com> | 2014-03-15 18:59:48 -0430 |
|---|---|---|
| committer | Paul Kehrer <paul.l.kehrer@gmail.com> | 2014-03-15 18:59:48 -0430 |
| commit | 3fb221f1fb02ffed7a558bd06ba41bb75c329fc5 (patch) | |
| tree | b2cc0d0e509569ec1deef80af6852a708e6a42d7 /cryptography | |
| parent | 106cc54b4344cc97af54eccb4d873434efe4203a (diff) | |
| parent | 93d7737fe3ef26c9b6b09a951826110e452b939d (diff) | |
| download | cryptography-3fb221f1fb02ffed7a558bd06ba41bb75c329fc5.tar.gz cryptography-3fb221f1fb02ffed7a558bd06ba41bb75c329fc5.tar.bz2 cryptography-3fb221f1fb02ffed7a558bd06ba41bb75c329fc5.zip | |
Merge pull request #803 from alex/style
Don't put things with side effects in assert statements
Diffstat (limited to 'cryptography')
| -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 |