[trafficserver] branch master updated: TS-4511: ATS crashes when no_dns_just_forward is configured without parent proxies (#698)

[us...@apache.org]

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

ushachar pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  fd834d7   TS-4511: ATS crashes when no_dns_just_forward is configured without parent proxies (#698)
fd834d7 is described below

commit fd834d7e05e8e3aac587d200a6c4ce61c81a06c7
Author: Uri Shachar <ur...@fire.glass>
AuthorDate: Tue Jun 7 20:12:25 2016 +0300

    TS-4511: ATS crashes when no_dns_just_forward is configured without parent proxies (#698)
---
 proxy/http/HttpSM.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 4f275a7..1668db3 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -7178,7 +7178,8 @@ HttpSM::set_next_state()
       DebugSM("dns", "[HttpTransact::HandleRequest] Skipping DNS lookup, provided by plugin");
       call_transact_and_set_next_state(NULL);
       break;
-    } else if (t_state.dns_info.looking_up == HttpTransact::ORIGIN_SERVER && t_state.http_config_param->no_dns_forward_to_parent) {
+    } else if (t_state.dns_info.looking_up == HttpTransact::ORIGIN_SERVER && t_state.http_config_param->no_dns_forward_to_parent &&
+               t_state.parent_result.result != PARENT_UNDEFINED) {
       if (t_state.cop_test_page)
         ats_ip_copy(t_state.host_db_info.ip(), t_state.state_machine->ua_session->get_netvc()->get_local_addr());
 

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