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: net/cert/x509_certificate_unittest.cc

Issue 634033002: Check whether or not a certificate is self-signed. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Once again, this is *Manos*... the *Hands*... of *Fate* Created 6 years, 2 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
Index: net/cert/x509_certificate_unittest.cc
diff --git a/net/cert/x509_certificate_unittest.cc b/net/cert/x509_certificate_unittest.cc
index 339851042c62fbc7663165be8cf69f13d89662cd..6abc2bc99d05059e87729193b4b1d45c6598cb6b 100644
--- a/net/cert/x509_certificate_unittest.cc
+++ b/net/cert/x509_certificate_unittest.cc
@@ -714,6 +714,20 @@ TEST(X509CertificateTest, IsIssuedByEncoded) {
EXPECT_TRUE(google_cert->IsIssuedByEncoded(issuers));
}
+TEST(X509CertificateTest, IsSelfSigned) {
+ base::FilePath certs_dir = GetTestCertsDirectory();
+
+ scoped_refptr<X509Certificate> cert(
+ ImportCertFromFile(certs_dir, "mit.davidben.der"));
+ ASSERT_NE(static_cast<X509Certificate*>(NULL), cert.get());
+ EXPECT_FALSE(X509Certificate::IsSelfSigned(cert->os_cert_handle()));
+
+ scoped_refptr<X509Certificate> self_signed(
+ ImportCertFromFile(certs_dir, "aia-root.pem"));
+ ASSERT_NE(static_cast<X509Certificate*>(NULL), self_signed.get());
+ EXPECT_TRUE(X509Certificate::IsSelfSigned(self_signed->os_cert_handle()));
+}
+
TEST(X509CertificateTest, IsIssuedByEncodedWithIntermediates) {
static const unsigned char kPolicyRootDN[] = {
0x30, 0x1e, 0x31, 0x1c, 0x30, 0x1a, 0x06, 0x03, 0x55, 0x04, 0x03, 0x0c,

Powered by Google App Engine
This is Rietveld 408576698