Bug 1343008 - Fix combination of H2, tls1.3 early-data and not safe http methods. r=mcmanus
authorDragana Damjanovic <dd.mozilla@gmail.com>
Mon, 27 Feb 2017 15:02:47 -0500
changeset 394108 5dc23fac58010d61efe9733da71177afcaa2e0e8
parent 394107 e00f58c31a85ad03e6eccfa99926f75687041254
child 394109 9a9362c9fcd9cf626b6107b571a469a8e47d2cb4
push id1468
push userasasaki@mozilla.com
push dateMon, 05 Jun 2017 19:31:07 +0000
treeherdermozilla-release@0641fc6ee9d1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmcmanus
bugs1343008
milestone54.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1343008 - Fix combination of H2, tls1.3 early-data and not safe http methods. r=mcmanus
netwerk/protocol/http/Http2Stream.cpp
--- a/netwerk/protocol/http/Http2Stream.cpp
+++ b/netwerk/protocol/http/Http2Stream.cpp
@@ -1475,18 +1475,18 @@ Http2Stream::MapStreamToHttpConnection()
 // -----------------------------------------------------------------------------
 // mirror nsAHttpTransaction
 // -----------------------------------------------------------------------------
 
 bool
 Http2Stream::Do0RTT()
 {
   MOZ_ASSERT(mTransaction);
-  mAttempting0RTT = true;
-  return mTransaction->Do0RTT();
+  mAttempting0RTT = mTransaction->Do0RTT();
+  return mAttempting0RTT;
 }
 
 nsresult
 Http2Stream::Finish0RTT(bool aRestart, bool aAlpnChanged)
 {
   MOZ_ASSERT(mTransaction);
   mAttempting0RTT = false;
   return mTransaction->Finish0RTT(aRestart, aAlpnChanged);