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

Unified Diff: net/quic/crypto/crypto_handshake.cc

Issue 23019005: Clear the QUIC cached proof when a client receives a REJ w/o a PROF tag. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 4 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
« net/quic/crypto/crypto_handshake.h ('K') | « net/quic/crypto/crypto_handshake.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/crypto/crypto_handshake.cc
diff --git a/net/quic/crypto/crypto_handshake.cc b/net/quic/crypto/crypto_handshake.cc
index d6a76f9f51195005e52a385a06cfbb636265f153..ce854c82e840fe539f4ed67061a3b38985eacfff 100644
--- a/net/quic/crypto/crypto_handshake.cc
+++ b/net/quic/crypto/crypto_handshake.cc
@@ -470,6 +470,12 @@ void QuicCryptoClientConfig::CachedState::SetProof(const vector<string>& certs,
server_config_sig_ = signature.as_string();
}
+void QuicCryptoClientConfig::CachedState::ClearProof() {
+ SetProofInvalid();
+ certs_.clear();
+ server_config_sig_.clear();
+}
+
void QuicCryptoClientConfig::CachedState::SetProofValid() {
server_config_valid_ = true;
}
@@ -832,6 +838,8 @@ QuicErrorCode QuicCryptoClientConfig::ProcessRejection(
}
cached->SetProof(certs, proof);
+ } else {
+ cached->ClearProof();
}
return QUIC_NO_ERROR;
« net/quic/crypto/crypto_handshake.h ('K') | « net/quic/crypto/crypto_handshake.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698