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

Unified Diff: chromeos/network/onc/onc_signature.cc

Issue 11469026: Extending ONC validator's logging. Completing toplevel validation. (Closed) Base URL: http://git.chromium.org/chromium/src.git@add_error_handling_to_validator
Patch Set: Rebased. Created 8 years 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 | « chromeos/network/onc/onc_signature.h ('k') | chromeos/network/onc/onc_translator_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/network/onc/onc_signature.cc
diff --git a/chromeos/network/onc/onc_signature.cc b/chromeos/network/onc/onc_signature.cc
index f71328c378dececf980d1f6b6aa20ae365d2a947..974df42e32e27baf21746573e9dde3feed3b543f 100644
--- a/chromeos/network/onc/onc_signature.cc
+++ b/chromeos/network/onc/onc_signature.cc
@@ -28,12 +28,6 @@ const OncValueSignature kStringListSignature = {
const OncValueSignature kIPConfigListSignature = {
Value::TYPE_LIST, NULL, &kIPConfigSignature
};
-const OncValueSignature kCertificateListSignature = {
- Value::TYPE_LIST, NULL, &kCertificateSignature
-};
-const OncValueSignature kNetworkConfigurationListSignature = {
- Value::TYPE_LIST, NULL, &kNetworkConfigurationSignature
-};
const OncFieldSignature issuer_subject_pattern_fields[] = {
{ certificate::kCommonName, NULL, &kStringSignature },
@@ -236,10 +230,18 @@ const OncFieldSignature certificate_fields[] = {
{ NULL }
};
-const OncFieldSignature unencrypted_configuration_fields[] = {
+const OncFieldSignature toplevel_configuration_fields[] = {
{ kCertificates, NULL, &kCertificateListSignature },
{ kNetworkConfigurations, NULL, &kNetworkConfigurationListSignature },
{ kType, NULL, &kStringSignature },
+ { encrypted::kCipher, NULL, &kStringSignature },
+ { encrypted::kCiphertext, NULL, &kStringSignature },
+ { encrypted::kHMAC, NULL, &kStringSignature },
+ { encrypted::kHMACMethod, NULL, &kStringSignature },
+ { encrypted::kIV, NULL, &kStringSignature },
+ { encrypted::kIterations, NULL, &kIntegerSignature },
+ { encrypted::kSalt, NULL, &kStringSignature },
+ { encrypted::kStretch, NULL, &kStringSignature },
{ NULL }
};
@@ -293,8 +295,14 @@ const OncValueSignature kCertificateSignature = {
const OncValueSignature kNetworkConfigurationSignature = {
Value::TYPE_DICTIONARY, network_configuration_fields, NULL
};
-const OncValueSignature kUnencryptedConfigurationSignature = {
- Value::TYPE_DICTIONARY, unencrypted_configuration_fields, NULL
+const OncValueSignature kCertificateListSignature = {
+ Value::TYPE_LIST, NULL, &kCertificateSignature
+};
+const OncValueSignature kNetworkConfigurationListSignature = {
+ Value::TYPE_LIST, NULL, &kNetworkConfigurationSignature
+};
+const OncValueSignature kToplevelConfigurationSignature = {
+ Value::TYPE_DICTIONARY, toplevel_configuration_fields, NULL
};
const OncFieldSignature* GetFieldSignature(const OncValueSignature& signature,
« no previous file with comments | « chromeos/network/onc/onc_signature.h ('k') | chromeos/network/onc/onc_translator_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698