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

Unified Diff: net/third_party/nss/patches/falsestartnpn2.patch

Issue 10221018: net: don't set NSS options in a callback. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 years, 8 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/third_party/nss/patches/falsestartnpn2.patch
diff --git a/net/third_party/nss/patches/falsestartnpn2.patch b/net/third_party/nss/patches/falsestartnpn2.patch
new file mode 100644
index 0000000000000000000000000000000000000000..0e64d8bce09525c67914f89ff076a24c6a16bd37
--- /dev/null
+++ b/net/third_party/nss/patches/falsestartnpn2.patch
@@ -0,0 +1,12 @@
+diff --git a/net/third_party/nss/ssl/ssl3con.c b/net/third_party/nss/ssl/ssl3con.c
+index 5476fa5..f97065a 100644
+--- a/net/third_party/nss/ssl/ssl3con.c
++++ b/net/third_party/nss/ssl/ssl3con.c
+@@ -6108,6 +6108,7 @@ ssl3_CanFalseStart(sslSocket *ss) {
+ rv = ss->opt.enableFalseStart &&
+ !ss->sec.isServer &&
+ !ss->ssl3.hs.isResuming &&
++ ssl3_ExtensionNegotiated(ss, ssl_next_proto_nego_xtn) &&
+ ss->ssl3.cwSpec &&
+
+ /* An attacker can control the selected ciphersuite so we only wish to

Powered by Google App Engine
This is Rietveld 408576698