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/08/25 16:26:11 UTC

[trafficserver] branch 9.2.x updated: Fix the connection limit crash while using parents (#7602) (#9052)

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 53940a461 Fix the connection limit crash while using parents (#7602) (#9052)
53940a461 is described below

commit 53940a4615839fb08ef2b12d7593b23380e854b4
Author: Evan Zelkowitz <ez...@apache.org>
AuthorDate: Thu Aug 25 10:15:45 2022 -0600

    Fix the connection limit crash while using parents (#7602) (#9052)
    
    (cherry picked from commit e8db7c495127915c0fd3d7eb4d9daa6f1b2fa67d)
    
    Co-authored-by: Vijay Mamidi <vi...@yahoo.com>
    (cherry picked from commit 4e3263ff60d64b6ccb5f8516cbb360ce2e28ab2a)
---
 proxy/http/HttpSM.cc | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 67b7d931d..86f716c9c 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -5287,12 +5287,11 @@ HttpSM::do_http_server_open(bool raw)
       } else { // queue size is 0, always block.
         ct_state.blocked();
         HTTP_INCREMENT_DYN_STAT(http_origin_connections_throttled_stat);
+        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();
       }
 
-      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);
-
       return;
     } else {
       ct_state.Note_Unblocked(&t_state.txn_conf->outbound_conntrack, ccount, &t_state.current.server->dst_addr.sa);