You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2020/03/30 20:40:00 UTC

[trafficserver] branch 9.0.x updated: Fix tunnel crash

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

zwoop pushed a commit to branch 9.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.0.x by this push:
     new de9eb28  Fix tunnel crash
de9eb28 is described below

commit de9eb2890e0e793e83a008c506f954de283c366f
Author: Susan Hinrichs <sh...@verizonmedia.com>
AuthorDate: Thu Mar 26 21:56:00 2020 +0000

    Fix tunnel crash
    
    (cherry picked from commit ba868d88a3f2ce4c58e48e2b7b5878a45c2922c4)
---
 proxy/http/HttpTunnel.cc | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/proxy/http/HttpTunnel.cc b/proxy/http/HttpTunnel.cc
index 4e69d2a..2b7f820 100644
--- a/proxy/http/HttpTunnel.cc
+++ b/proxy/http/HttpTunnel.cc
@@ -906,6 +906,11 @@ HttpTunnel::producer_run(HttpTunnelProducer *p)
   for (c = p->consumer_list.head; c; c = c->link.next) {
     int64_t c_write = consumer_n;
 
+    // Don't bother to set up the consumer if it is dead
+    if (!c->alive) {
+      continue;
+    }
+
     if (!p->alive) {
       // Adjust the amount of chunked data to write if the only data was in the initial read
       // The amount to write in some cases is dependent on the type of the consumer, so this