commit ddf2d676648e3510a003e4a985782d5b51b41b0b
author Shawn Willden <swillden@google.com> Fri Feb 20 16:37:12 2015 -0700
committer Shawn Willden <swillden@google.com> Fri Feb 20 16:37:12 2015 -0700
tree 753b2019d7dc7945d8ef1b4f1228e71828a07e7c
parent a278f6156b9e57e1de8ca380b14fef113ad10be8

Check return codes in softkeymaster.

Also add some more logging and a test for ECDSA-192.

Change-Id: Icc4327bb1f5cb14bc22a63e80592e29a349f1bc2

google_keymaster_test.cpp

diff --git a/google_keymaster_test.cpp b/google_keymaster_test.cpp
index 91aef7b..f16b6f8 100644
--- a/google_keymaster_test.cpp
+++ b/google_keymaster_test.cpp
@@ -533,8 +533,7 @@
     free(key_blob);
 }
 
-TEST_F(OldKeyGeneration, Ecdsa) {
-
+TEST_F(OldKeyGeneration, Ecdsa256) {
     keymaster_ec_keygen_params_t params = {.field_size = 256};
     uint8_t* key_blob;
     size_t key_blob_length;
@@ -545,6 +544,14 @@
     free(key_blob);
 }
 
+TEST_F(OldKeyGeneration, Ecdsa192Fails) {
+    keymaster_ec_keygen_params_t params = {.field_size = 192};
+    uint8_t* key_blob;
+    size_t key_blob_length;
+    EXPECT_EQ(KM_ERROR_UNSUPPORTED_KEY_SIZE,
+              device()->generate_keypair(device(), TYPE_EC, &params, &key_blob, &key_blob_length));
+}
+
 class NewKeyGeneration : public KeymasterTest {
   protected:
     void CheckBaseParams() {