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

Side by Side Diff: chromeos/network/onc/onc_certificate_importer_impl.h

Issue 148183013: Use per-user nssdb in onc certificate importer (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: should be ready Created 6 years, 10 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_ 5 #ifndef CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_
6 #define CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_ 6 #define CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <string> 9 #include <string>
10 #include <vector> 10 #include <vector>
(...skipping 22 matching lines...) Expand all
33 // imports) into the certificate store. The GUID of Client certificates is 33 // imports) into the certificate store. The GUID of Client certificates is
34 // stored together with the certificate as Nickname. In contrast, Server and CA 34 // stored together with the certificate as Nickname. In contrast, Server and CA
35 // certificates are identified by their PEM and not by GUID. 35 // certificates are identified by their PEM and not by GUID.
36 // TODO(pneubeck): Replace Nickname by PEM for Client 36 // TODO(pneubeck): Replace Nickname by PEM for Client
37 // certificates. http://crbug.com/252119 37 // certificates. http://crbug.com/252119
38 class CHROMEOS_EXPORT CertificateImporterImpl : public CertificateImporter { 38 class CHROMEOS_EXPORT CertificateImporterImpl : public CertificateImporter {
39 public: 39 public:
40 typedef std::map<std::string, scoped_refptr<net::X509Certificate> > 40 typedef std::map<std::string, scoped_refptr<net::X509Certificate> >
41 CertsByGUID; 41 CertsByGUID;
42 42
43 CertificateImporterImpl(); 43 explicit CertificateImporterImpl(net::NSSCertDatabase* target_nssdb_);
44 44
45 // CertificateImporter overrides 45 // CertificateImporter overrides
46 virtual bool ImportCertificates( 46 virtual bool ImportCertificates(
47 const base::ListValue& certificates, 47 const base::ListValue& certificates,
48 ::onc::ONCSource source, 48 ::onc::ONCSource source,
49 net::CertificateList* onc_trusted_certificates) OVERRIDE; 49 net::CertificateList* onc_trusted_certificates) OVERRIDE;
50 50
51 // This implements ImportCertificates. Additionally, if 51 // This implements ImportCertificates. Additionally, if
52 // |imported_server_and_ca_certs| is not NULL, it will be filled with the 52 // |imported_server_and_ca_certs| is not NULL, it will be filled with the
53 // (GUID, Certificate) pairs of all succesfully imported Server and CA 53 // (GUID, Certificate) pairs of all succesfully imported Server and CA
54 // certificates. 54 // certificates.
55 bool ParseAndStoreCertificates(bool allow_trust_imports, 55 bool ParseAndStoreCertificates(bool allow_trust_imports,
56 const base::ListValue& onc_certificates, 56 const base::ListValue& onc_certificates,
57 net::CertificateList* onc_trusted_certificates, 57 net::CertificateList* onc_trusted_certificates,
58 CertsByGUID* imported_server_and_ca_certs); 58 CertsByGUID* imported_server_and_ca_certs);
59 59
60 private:
60 // Lists the certificates that have the string |label| as their certificate 61 // Lists the certificates that have the string |label| as their certificate
61 // nickname (exact match). 62 // nickname (exact match).
62 static void ListCertsWithNickname(const std::string& label, 63 static void ListCertsWithNickname(const std::string& label,
63 net::CertificateList* result); 64 net::CertificateList* result,
65 net::NSSCertDatabase* target_nssdb);
64 66
65 private:
66 // Deletes any certificate that has the string |label| as its nickname (exact 67 // Deletes any certificate that has the string |label| as its nickname (exact
67 // match). 68 // match).
68 static bool DeleteCertAndKeyByNickname(const std::string& label); 69 static bool DeleteCertAndKeyByNickname(const std::string& label,
70 net::NSSCertDatabase* target_nssdb);
69 71
70 // Parses and stores/removes |certificate| in/from the certificate 72 // Parses and stores/removes |certificate| in/from the certificate
71 // store. Returns true if the operation succeeded. 73 // store. Returns true if the operation succeeded.
72 bool ParseAndStoreCertificate( 74 bool ParseAndStoreCertificate(
73 bool allow_trust_imports, 75 bool allow_trust_imports,
74 const base::DictionaryValue& certificate, 76 const base::DictionaryValue& certificate,
75 net::CertificateList* onc_trusted_certificates, 77 net::CertificateList* onc_trusted_certificates,
76 CertsByGUID* imported_server_and_ca_certs); 78 CertsByGUID* imported_server_and_ca_certs);
77 79
78 // Imports the Server or CA certificate |certificate|. Web trust is only 80 // Imports the Server or CA certificate |certificate|. Web trust is only
79 // applied if the certificate requests the TrustBits attribute "Web" and if 81 // applied if the certificate requests the TrustBits attribute "Web" and if
80 // the |allow_trust_imports| permission is granted, otherwise the attribute is 82 // the |allow_trust_imports| permission is granted, otherwise the attribute is
81 // ignored. 83 // ignored.
82 bool ParseServerOrCaCertificate( 84 bool ParseServerOrCaCertificate(
83 bool allow_trust_imports, 85 bool allow_trust_imports,
84 const std::string& cert_type, 86 const std::string& cert_type,
85 const std::string& guid, 87 const std::string& guid,
86 const base::DictionaryValue& certificate, 88 const base::DictionaryValue& certificate,
87 net::CertificateList* onc_trusted_certificates, 89 net::CertificateList* onc_trusted_certificates,
88 CertsByGUID* imported_server_and_ca_certs); 90 CertsByGUID* imported_server_and_ca_certs);
89 91
90 bool ParseClientCertificate(const std::string& guid, 92 bool ParseClientCertificate(const std::string& guid,
91 const base::DictionaryValue& certificate); 93 const base::DictionaryValue& certificate);
92 94
95 // The certificate database to which certificates are imported.
96 net::NSSCertDatabase* target_nssdb_;
97
93 DISALLOW_COPY_AND_ASSIGN(CertificateImporterImpl); 98 DISALLOW_COPY_AND_ASSIGN(CertificateImporterImpl);
94 }; 99 };
95 100
96 } // namespace onc 101 } // namespace onc
97 } // namespace chromeos 102 } // namespace chromeos
98 103
99 #endif // CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_ 104 #endif // CHROMEOS_NETWORK_ONC_ONC_CERTIFICATE_IMPORTER_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698