commit 289e76b6ff2b3f07f73166df2f7c7877b20f2d43
author David Benjamin <davidben@google.com> Thu Jun 15 17:07:05 2017 -0400
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> Fri Jun 16 13:45:44 2017 +0000
tree 665a305932bc810735b492236deb6a40bb395c5e
parent 0391f16673cb8da72f9944880e0599282463131b

EVP_PKEY_cmp does not return a boolean.

Change-Id: Ic82b093d006cd3c8c5a762d04c86bbcefeee3b24
Reviewed-on: https://boringssl-review.googlesource.com/17205
Commit-Queue: Steven Valdez <svaldez@google.com>
Reviewed-by: Steven Valdez <svaldez@google.com>
CQ-Verified: CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org>

crypto/evp/evp_extra_test.cc

diff --git a/crypto/evp/evp_extra_test.cc b/crypto/evp/evp_extra_test.cc
index 2698663..f12650e 100644
--- a/crypto/evp/evp_extra_test.cc
+++ b/crypto/evp/evp_extra_test.cc
@@ -614,13 +614,13 @@
   EXPECT_EQ(Bytes(kPrivateKeyPKCS8), Bytes(der, der_len));
 
   // Test EVP_PKEY_cmp.
-  EXPECT_TRUE(EVP_PKEY_cmp(pubkey.get(), privkey.get()));
+  EXPECT_EQ(1, EVP_PKEY_cmp(pubkey.get(), privkey.get()));
 
   static const uint8_t kZeros[32] = {0};
   bssl::UniquePtr<EVP_PKEY> pubkey2(EVP_PKEY_new_ed25519_public(kZeros));
   ASSERT_TRUE(pubkey2);
-  EXPECT_FALSE(EVP_PKEY_cmp(pubkey.get(), pubkey2.get()));
-  EXPECT_FALSE(EVP_PKEY_cmp(privkey.get(), pubkey2.get()));
+  EXPECT_EQ(0, EVP_PKEY_cmp(pubkey.get(), pubkey2.get()));
+  EXPECT_EQ(0, EVP_PKEY_cmp(privkey.get(), pubkey2.get()));
 
   // Ed25519 may not be used streaming.
   bssl::ScopedEVP_MD_CTX ctx;