Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1504)

Unified Diff: content/child/webcrypto/test/aes_ctr_unittest.cc

Issue 491763002: [webcrypto] Implement AES-CTR using BoringSSL. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase onto master (moves unittest to its own file) Created 6 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/child/webcrypto/status.cc ('k') | content/content_child.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/child/webcrypto/test/aes_ctr_unittest.cc
diff --git a/content/child/webcrypto/test/aes_ctr_unittest.cc b/content/child/webcrypto/test/aes_ctr_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..497459952db6680fbe2ea6315037eba7d4b01d61
--- /dev/null
+++ b/content/child/webcrypto/test/aes_ctr_unittest.cc
@@ -0,0 +1,218 @@
+// Copyright 2014 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "base/stl_util.h"
+#include "content/child/webcrypto/algorithm_dispatch.h"
+#include "content/child/webcrypto/crypto_data.h"
+#include "content/child/webcrypto/status.h"
+#include "content/child/webcrypto/test/test_helpers.h"
+#include "content/child/webcrypto/webcrypto_util.h"
+#include "third_party/WebKit/public/platform/WebCryptoAlgorithmParams.h"
+#include "third_party/WebKit/public/platform/WebCryptoKeyAlgorithm.h"
+
+namespace content {
+
+namespace webcrypto {
+
+namespace {
+
+bool SupportsAesCtr() {
+#if defined(USE_OPENSSL)
+ return true;
+#else
+ return false;
+#endif
+}
+
+// Creates an AES-CTR algorithm for encryption/decryption.
+blink::WebCryptoAlgorithm CreateAesCtrAlgorithm(
+ const std::vector<uint8_t>& counter,
+ uint8_t length_bits) {
+ return blink::WebCryptoAlgorithm::adoptParamsAndCreate(
+ blink::WebCryptoAlgorithmIdAesCtr,
+ new blink::WebCryptoAesCtrParams(
+ length_bits, vector_as_array(&counter), counter.size()));
+}
+
+TEST(WebCryptoAesCtrTest, EncryptDecryptKnownAnswer) {
+ if (!SupportsAesCtr()) {
+ LOG(WARNING) << "Skipping test because AES-CTR is not supported";
+ return;
+ }
+
+ scoped_ptr<base::ListValue> tests;
+ ASSERT_TRUE(ReadJsonTestFileToList("aes_ctr.json", &tests));
+
+ for (size_t test_index = 0; test_index < tests->GetSize(); ++test_index) {
+ SCOPED_TRACE(test_index);
+ base::DictionaryValue* test;
+ ASSERT_TRUE(tests->GetDictionary(test_index, &test));
+
+ std::vector<uint8_t> test_key = GetBytesFromHexString(test, "key");
+ std::vector<uint8_t> test_counter = GetBytesFromHexString(test, "counter");
+ int counter_length_bits = 0;
+ ASSERT_TRUE(test->GetInteger("length", &counter_length_bits));
+
+ std::vector<uint8_t> test_plain_text =
+ GetBytesFromHexString(test, "plain_text");
+ std::vector<uint8_t> test_cipher_text =
+ GetBytesFromHexString(test, "cipher_text");
+
+ blink::WebCryptoKey key = ImportSecretKeyFromRaw(
+ test_key,
+ CreateAlgorithm(blink::WebCryptoAlgorithmIdAesCtr),
+ blink::WebCryptoKeyUsageEncrypt | blink::WebCryptoKeyUsageDecrypt);
+
+ EXPECT_EQ(test_key.size() * 8, key.algorithm().aesParams()->lengthBits());
+
+ std::vector<uint8_t> output;
+
+ // Test encryption.
+ EXPECT_EQ(Status::Success(),
+ Encrypt(CreateAesCtrAlgorithm(test_counter, counter_length_bits),
+ key,
+ CryptoData(test_plain_text),
+ &output));
+ EXPECT_BYTES_EQ(test_cipher_text, output);
+
+ // Test decryption.
+ EXPECT_EQ(Status::Success(),
+ Decrypt(CreateAesCtrAlgorithm(test_counter, counter_length_bits),
+ key,
+ CryptoData(test_cipher_text),
+ &output));
+ EXPECT_BYTES_EQ(test_plain_text, output);
+ }
+}
+
+// The counter block must be exactly 16 bytes.
+TEST(WebCryptoAesCtrTest, InvalidCounterBlockLength) {
+ if (!SupportsAesCtr()) {
+ LOG(WARNING) << "Skipping test because AES-CTR is not supported";
+ return;
+ }
+
+ const unsigned int kBadCounterBlockLengthBytes[] = {0, 15, 17};
+
+ blink::WebCryptoKey key = ImportSecretKeyFromRaw(
+ std::vector<uint8>(16), // 128-bit key of all zeros.
+ CreateAlgorithm(blink::WebCryptoAlgorithmIdAesCtr),
+ blink::WebCryptoKeyUsageEncrypt | blink::WebCryptoKeyUsageDecrypt);
+
+ std::vector<uint8_t> input(32);
+ std::vector<uint8_t> output;
+
+ for (size_t i = 0; i < arraysize(kBadCounterBlockLengthBytes); ++i) {
+ std::vector<uint8_t> bad_counter(kBadCounterBlockLengthBytes[i]);
+
+ EXPECT_EQ(Status::ErrorIncorrectSizeAesCtrCounter(),
+ Encrypt(CreateAesCtrAlgorithm(bad_counter, 128),
+ key,
+ CryptoData(input),
+ &output));
+
+ EXPECT_EQ(Status::ErrorIncorrectSizeAesCtrCounter(),
+ Decrypt(CreateAesCtrAlgorithm(bad_counter, 128),
+ key,
+ CryptoData(input),
+ &output));
+ }
+}
+
+// The counter length cannot be less than 1 or greater than 128.
+TEST(WebCryptoAesCtrTest, InvalidCounterLength) {
+ if (!SupportsAesCtr()) {
+ LOG(WARNING) << "Skipping test because AES-CTR is not supported";
+ return;
+ }
+
+ const uint8_t kBadCounterLengthBits[] = {0, 129};
+
+ blink::WebCryptoKey key = ImportSecretKeyFromRaw(
+ std::vector<uint8>(16), // 128-bit key of all zeros.
+ CreateAlgorithm(blink::WebCryptoAlgorithmIdAesCtr),
+ blink::WebCryptoKeyUsageEncrypt | blink::WebCryptoKeyUsageDecrypt);
+
+ std::vector<uint8_t> counter(16);
+ std::vector<uint8_t> input(32);
+ std::vector<uint8_t> output;
+
+ for (size_t i = 0; i < arraysize(kBadCounterLengthBits); ++i) {
+ uint8_t bad_counter_length_bits = kBadCounterLengthBits[i];
+
+ EXPECT_EQ(Status::ErrorInvalidAesCtrCounterLength(),
+ Encrypt(CreateAesCtrAlgorithm(counter, bad_counter_length_bits),
+ key,
+ CryptoData(input),
+ &output));
+
+ EXPECT_EQ(Status::ErrorInvalidAesCtrCounterLength(),
+ Decrypt(CreateAesCtrAlgorithm(counter, bad_counter_length_bits),
+ key,
+ CryptoData(input),
+ &output));
+ }
+}
+
+// Tests wrap-around using a 4-bit counter.
+//
+// Wrap-around is allowed, however if the counter repeats itself an error should
+// be thrown.
+//
+// Using a 4-bit counter it is possible to encrypt 16 blocks. However the 17th
+// block would end up wrapping back to the starting value.
+TEST(WebCryptoAesCtrTest, OverflowAndRepeatCounter) {
+ if (!SupportsAesCtr()) {
+ LOG(WARNING) << "Skipping test because AES-CTR is not supported";
+ return;
+ }
+
+ const uint8_t kCounterLengthBits = 4;
+ const uint8_t kStartCounter[] = {0, 1, 15};
+
+ blink::WebCryptoKey key = ImportSecretKeyFromRaw(
+ std::vector<uint8>(16), // 128-bit key of all zeros.
+ CreateAlgorithm(blink::WebCryptoAlgorithmIdAesCtr),
+ blink::WebCryptoKeyUsageEncrypt | blink::WebCryptoKeyUsageDecrypt);
+
+ std::vector<uint8_t> buffer(272);
+
+ // 16 and 17 AES blocks worth of data respectively (AES blocks are 16 bytes
+ // long).
+ CryptoData input_16(vector_as_array(&buffer), 256);
+ CryptoData input_17(vector_as_array(&buffer), 272);
+
+ std::vector<uint8_t> output;
+
+ for (size_t i = 0; i < arraysize(kStartCounter); ++i) {
+ std::vector<uint8_t> counter(16);
+ counter[15] = kStartCounter[i];
+
+ // Baseline test: Encrypting 16 blocks should work (don't bother to check
+ // output, the known answer tests already do that).
+ EXPECT_EQ(Status::Success(),
+ Encrypt(CreateAesCtrAlgorithm(counter, kCounterLengthBits),
+ key,
+ input_16,
+ &output));
+
+ // Encrypting/Decrypting 17 however should fail.
+ EXPECT_EQ(Status::ErrorAesCtrInputTooLongCounterRepeated(),
+ Encrypt(CreateAesCtrAlgorithm(counter, kCounterLengthBits),
+ key,
+ input_17,
+ &output));
+ EXPECT_EQ(Status::ErrorAesCtrInputTooLongCounterRepeated(),
+ Decrypt(CreateAesCtrAlgorithm(counter, kCounterLengthBits),
+ key,
+ input_17,
+ &output));
+ }
+}
+
+} // namespace
+
+} // namespace webcrypto
+
+} // namespace content
« no previous file with comments | « content/child/webcrypto/status.cc ('k') | content/content_child.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698