Fixed EC Attributes returned and enabled test cases for this.

Author: frank-zonn

pkcs11/util/ec.py

@@ -81,8 +81,8 @@ def decode_ec_private_key(der):
     return {
         Attribute.KEY_TYPE: KeyType.EC,
         Attribute.CLASS: ObjectClass.PRIVATE_KEY,
-        Attribute.EC_PARAMS: asn1['parameters'].dump(),
-        Attribute.VALUE: asn1['private_key'],
+        Attribute.EC_PARAMS: asn1['parameters'].chosen.dump(),
+        Attribute.VALUE: asn1['private_key'].contents,
     }
 
 

tests/test_ecc.py

@@ -143,10 +143,9 @@ def test_import_key_pair(self):
         """)
         pub = self.session.create_object(decode_ec_public_key(pub))
 
-        # FIXME: why can't I sign this?
-        # signature = priv.sign(b'Example', mechanism=Mechanism.ECDSA)
-        # self.assertTrue(pub.verify(b'Example', signature,
-        #                            mechanism=Mechanism.ECDSA))
+        signature = priv.sign(b'Example', mechanism=Mechanism.ECDSA)
+        self.assertTrue(pub.verify(b'Example', signature,
+                                   mechanism=Mechanism.ECDSA))
 
     @requires(Mechanism.EC_EDWARDS_KEY_PAIR_GEN, Mechanism.EDDSA)
     def test_sign_eddsa(self):
@@ -162,4 +161,3 @@ def test_sign_eddsa(self):
         data = b'HI BOB!'
         eddsa = priv.sign(data, mechanism=mechanism)
         self.assertTrue(pub.verify(data, eddsa, mechanism=mechanism))
-