[trafficserver] branch master updated: Fix crash from HttpSM::setup_100_continue_transfer

[sh...@apache.org]

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

shinrich pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new 7a3ab0c  Fix crash from HttpSM::setup_100_continue_transfer
7a3ab0c is described below

commit 7a3ab0ca39109d67cafadac6754c55a8724f9b25
Author: Susan Hinrichs <sh...@apache.org>
AuthorDate: Mon Nov 27 16:46:46 2017 -0600

    Fix crash from HttpSM::setup_100_continue_transfer
---
 proxy/http/HttpSM.cc | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 64affa1..5db5ad2 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -6141,6 +6141,10 @@ HttpSM::setup_100_continue_transfer()
 
   HTTP_SM_SET_DEFAULT_HANDLER(&HttpSM::tunnel_handler_100_continue);
 
+  // Clear the decks before we set up new producers.  As things stand, we cannot have two static operators
+  // at once
+  tunnel.reset();
+
   // Setup the tunnel to the client
   HttpTunnelProducer *p = tunnel.add_producer(HTTP_TUNNEL_STATIC_PRODUCER, client_response_hdr_bytes, buf_start,
                                               (HttpProducerHandler) nullptr, HT_STATIC, "internal msg - 100 continue");

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