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

Unified Diff: chrome/browser/chromeos/network_settings/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: Initial patch. 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
Index: chrome/browser/chromeos/network_settings/onc_signature.cc
diff --git a/chrome/browser/chromeos/network_settings/onc_signature.cc b/chrome/browser/chromeos/network_settings/onc_signature.cc
index ee8cf524c6b92c8a01b8a8a90678dfd0194fd389..29e03c6aae54c9659c81564b335bb1d11583c99e 100644
--- a/chrome/browser/chromeos/network_settings/onc_signature.cc
+++ b/chrome/browser/chromeos/network_settings/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,

Powered by Google App Engine
This is Rietveld 408576698