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 2019/01/30 07:34:24 UTC
[trafficserver] branch 7.1.x updated: Fixes unmatched port when
looking up HostDB
This is an automated email from the ASF dual-hosted git repository.
zwoop pushed a commit to branch 7.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git
The following commit(s) were added to refs/heads/7.1.x by this push:
new 9784c57 Fixes unmatched port when looking up HostDB
9784c57 is described below
commit 9784c572fc19ddc484dae4f83718bc5af0524db4
Author: scw00 <sc...@apache.org>
AuthorDate: Wed Jan 16 15:01:40 2019 +0800
Fixes unmatched port when looking up HostDB
(cherry picked from commit 646b2d9802202dee850f52a177bb6f6ef870c65b)
---
proxy/http/HttpSM.cc | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index f9ec9f8..ebdc6b9 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -4120,10 +4120,14 @@ HttpSM::do_hostdb_lookup()
// If there is not a current server, we must be looking up the origin
// server at the beginning of the transaction
- int server_port = t_state.current.server ?
- t_state.current.server->dst_addr.host_order_port() :
- t_state.server_info.dst_addr.isValid() ? t_state.server_info.dst_addr.host_order_port() :
- t_state.hdr_info.client_request.port_get();
+ int server_port = 0;
+ if (t_state.current.server && t_state.current.server->dst_addr.isValid()) {
+ server_port = t_state.current.server->dst_addr.host_order_port();
+ } else if (t_state.server_info.dst_addr.isValid()) {
+ server_port = t_state.server_info.dst_addr.host_order_port();
+ } else {
+ server_port = t_state.hdr_info.client_request.port_get();
+ }
if (t_state.api_txn_dns_timeout_value != -1) {
DebugSM("http_timeout", "beginning DNS lookup. allowing %d mseconds for DNS lookup", t_state.api_txn_dns_timeout_value);