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

Unified Diff: net/http/transport_security_state.cc

Issue 1580583002: Add a whitelist for QUIC hosts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix comments Created 4 years, 11 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/http/transport_security_state.cc
diff --git a/net/http/transport_security_state.cc b/net/http/transport_security_state.cc
index e77c8c5e2138a4688a03a9d76047d9235e88dbc4..3a38992067e8b4dcb919b4fbfcca70b5156bfb2f 100644
--- a/net/http/transport_security_state.cc
+++ b/net/http/transport_security_state.cc
@@ -1100,6 +1100,22 @@ bool TransportSecurityState::GetStaticDomainState(const std::string& host,
return true;
}
+bool TransportSecurityState::IsGooglePinnedHost(const std::string& host) const {
+ DCHECK(CalledOnValidThread());
+
+ if (!IsBuildTimely())
+ return false;
+
+ PreloadResult result;
+ if (!DecodeHSTSPreload(host, &result))
+ return false;
+
+ if (!result.has_pins)
Ryan Sleevi 2016/01/12 00:29:39 BUG? You should be checking enable_static_pins_, r
Ryan Hamilton 2016/01/12 01:21:24 I think this is a feature. In the initial version
+ return false;
+
+ return kPinsets[result.pinset_id].accepted_pins == kGoogleAcceptableCerts;
Ryan Sleevi 2016/01/12 00:29:39 BUG? The original code defends against corruption
Ryan Hamilton 2016/01/12 01:21:24 It's defending against corruption? Interesting. I
+}
+
bool TransportSecurityState::GetStaticExpectCTState(
const std::string& host,
ExpectCTState* expect_ct_state) const {

Powered by Google App Engine
This is Rietveld 408576698