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 2022/07/26 22:30:30 UTC

[trafficserver] branch 9.2.x updated: Fix reverting PR#7302 (#8975)

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

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


The following commit(s) were added to refs/heads/9.2.x by this push:
     new 9348455ba Fix reverting PR#7302 (#8975)
9348455ba is described below

commit 9348455ba6bcd29512a8839cdba9aa40b799160f
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Fri Jul 22 07:29:20 2022 +0900

    Fix reverting PR#7302 (#8975)
    
    PR#7302 was reverted by PR#8316 as an incompatible change for 9.2.0.
    It looks like the revert commit has a mistake that made a crash by
    calling `HttpSM::send_origin_throttled_response()` twice.
    
    (cherry picked from commit 3cccd2d94d73fe9aa8452adaa9fdc30ad9215e84)
---
 proxy/http/HttpSM.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index e1bd524c5..84a7ccf50 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -5313,7 +5313,6 @@ HttpSM::do_http_server_open(bool raw)
 
       ct_state.Warn_Blocked(&t_state.txn_conf->outbound_conntrack, sm_id, ccount - 1, &t_state.current.server->dst_addr.sa,
                             debug_on && is_debug_tag_set("http") ? "http" : nullptr);
-      send_origin_throttled_response();
 
       return;
     } else {