You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by so...@apache.org on 2016/06/01 21:00:53 UTC

[trafficserver] 02/02: TS-4489: Remove second calling of Http2ConnectionState::delete_stream in Http2Stream::do_io_close

This is an automated email from the ASF dual-hosted git repository.

sorber pushed a commit to branch 6.2.x
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

commit 5be1ebdc0922b1d47aaba9e522f18713a9d73cda
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Tue May 31 12:18:34 2016 -0700

    TS-4489: Remove second calling of Http2ConnectionState::delete_stream in Http2Stream::do_io_close
    
    (cherry picked from commit f1642f0c9f68584a2e001d910c0589bba124dfbc)
---
 proxy/http2/Http2Stream.cc | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/proxy/http2/Http2Stream.cc b/proxy/http2/Http2Stream.cc
index 42f6659..f062b97 100644
--- a/proxy/http2/Http2Stream.cc
+++ b/proxy/http2/Http2Stream.cc
@@ -255,10 +255,8 @@ Http2Stream::do_io_close(int /* flags */)
 
     if (parent) {
       // Make sure any trailing end of stream frames are sent
+      // Ourselve will be removed at send_data_frames or closing connection phase
       static_cast<Http2ClientSession *>(parent)->connection_state.send_data_frames(this);
-
-      // Remove ourselves from the stream list
-      static_cast<Http2ClientSession *>(parent)->connection_state.delete_stream(this);
     }
     parent = NULL;
 

-- 
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>.