aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorPaul Kehrer <paul.l.kehrer@gmail.com>2014-12-12 11:50:44 -0600
committerPaul Kehrer <paul.l.kehrer@gmail.com>2014-12-15 15:49:50 -0600
commit6c660a88f1ed6d03968b26328a285cfecc4c9a2c (patch)
tree3870aa7d327c954b2ef645972fa2360323bad80f /src
parentd273adeae286186a71d38856dc2c2cea4acd378a (diff)
downloadcryptography-6c660a88f1ed6d03968b26328a285cfecc4c9a2c.tar.gz
cryptography-6c660a88f1ed6d03968b26328a285cfecc4c9a2c.tar.bz2
cryptography-6c660a88f1ed6d03968b26328a285cfecc4c9a2c.zip
raise error on unnamed EC curve certificates when calling public_key
...for now
Diffstat (limited to 'src')
-rw-r--r--src/cryptography/hazmat/backends/openssl/x509.py20
1 files changed, 20 insertions, 0 deletions
diff --git a/src/cryptography/hazmat/backends/openssl/x509.py b/src/cryptography/hazmat/backends/openssl/x509.py
index fa4d1a01..1de3f4d5 100644
--- a/src/cryptography/hazmat/backends/openssl/x509.py
+++ b/src/cryptography/hazmat/backends/openssl/x509.py
@@ -63,6 +63,26 @@ class _X509Certificate(object):
pkey = self._backend._lib.X509_get_pubkey(self._x509)
assert pkey != self._backend._ffi.NULL
pkey = self._backend._ffi.gc(pkey, self._backend._lib.EVP_PKEY_free)
+ # The following check is to find ECDSA certificates with unnamed
+ # curves and raise an error for now.
+ if (
+ self._backend._lib.Cryptography_HAS_EC == 1 and
+ pkey.type == self._backend._lib.EVP_PKEY_EC
+ ):
+ ec_cdata = self._backend._lib.EVP_PKEY_get1_EC_KEY(pkey)
+ assert ec_cdata != self._backend._ffi.NULL
+ ec_cdata = self._backend._ffi.gc(
+ ec_cdata, self._backend._lib.EC_KEY_free
+ )
+ group = self._backend._lib.EC_KEY_get0_group(ec_cdata)
+ assert group != self._backend._ffi.NULL
+ nid = self._backend._lib.EC_GROUP_get_curve_name(group)
+ if nid == self._backend._lib.NID_undef:
+ raise NotImplementedError(
+ "ECDSA certificates without named curves are unsupported "
+ "at this time"
+ )
+
return self._backend._evp_pkey_to_public_key(pkey)
@property