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

Unified Diff: net/quic/core/crypto/quic_crypto_server_config.cc

Issue 2324833004: Define Stable API for WebRTC/Quartc (Closed)
Patch Set: Add virtual destructors required by WebRTC compiler. Created 4 years, 3 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/quic/core/crypto/quic_crypto_server_config.cc
diff --git a/net/quic/core/crypto/quic_crypto_server_config.cc b/net/quic/core/crypto/quic_crypto_server_config.cc
index 66860e767845e950f2a746a03facbc120505d25d..ba3b6fc40592319c88882a0ffec66d1bf2e57e99 100644
--- a/net/quic/core/crypto/quic_crypto_server_config.cc
+++ b/net/quic/core/crypto/quic_crypto_server_config.cc
@@ -630,6 +630,7 @@ QuicErrorCode QuicCryptoServerConfig::ProcessClientHello(
primary_config->serialized, version, chlo_hash,
&crypto_proof->chain, &crypto_proof->signature,
&crypto_proof->cert_sct)) {
+ *error_details = "Missing or invalid crypto proof.";
Ryan Hamilton 2016/10/10 19:48:30 nit: This should probably land in a distinct CL an
zhihuang1 2016/10/13 06:22:39 Done. https://codereview.chromium.org/2302643003/
return QUIC_HANDSHAKE_FAILED;
}

Powered by Google App Engine
This is Rietveld 408576698