Allow empty keys in hmac_openssl.cc.

crypto/hmac_unittest.cc

Index: crypto/hmac_unittest.cc
diff --git a/crypto/hmac_unittest.cc b/crypto/hmac_unittest.cc
index f0844a9186c9d661273483266408dcb9b2e1a3c1..bac3e525314d5791ad8f84c3baf126d61b8d8147 100644
--- a/crypto/hmac_unittest.cc
+++ b/crypto/hmac_unittest.cc
@@ -275,3 +275,20 @@ TEST(HMACTest, Verify) {
         base::StringPiece(empty_digest, kSHA1DigestSize)));
   }
 }
+
+TEST(HMACTest, EmptyKey) {
+  // Test vector from https://en.wikipedia.org/wiki/HMAC
+  const char* kExpectedDigest =
+      "\xFB\xDB\x1D\x1B\x18\xAA\x6C\x08\x32\x4B\x7D\x64\xB7\x1F\xB7\x63"
+      "\x70\x69\x0E\x1D";
+
+  crypto::HMAC hmac(crypto::HMAC::SHA1);
+  ASSERT_TRUE(hmac.Init(NULL, 0));
+
+  unsigned char digest[kSHA1DigestSize];
+  EXPECT_TRUE(hmac.Sign(base::StringPiece("", 0), digest, kSHA1DigestSize));
+  EXPECT_EQ(0, memcmp(kExpectedDigest, digest, kSHA1DigestSize));
+
+  EXPECT_TRUE(hmac.Verify(
+      "", base::StringPiece(kExpectedDigest, kSHA1DigestSize)));

[wtc, 2014/04/09 22:10:49]

We should also pass a StringPiece here. I suggest we use a local variable:

  base::StringPiece key("", 0);

and pass |key| to hmac.Sign() and hmac.Verify().

[davidben, 2014/04/09 23:01:52]

Oh, missed that one. Done.

+}