[TS-4948] CID 1364117 (Forward NULL) in proxy/http/HttpSM.cc

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 7.1.0
• Component/s: HTTP
• Assignee: Leif Hedstrom
• Reporter: Leif Hedstrom
• Created: 11/Oct/16 15:10
• Updated: 06/Jan/17 16:40
• Resolved: 06/Jan/17 16:40

Description

*** CID 1364117:    (FORWARD_NULL)
/proxy/http/HttpSM.cc: 2111 in HttpSM::process_hostdb_info(HostDBInfo *)()
2105     void
2106     HttpSM::process_hostdb_info(HostDBInfo *r)
2107     {
2108       // Increment the refcount to our item, since we are pointing at it
2109       t_state.hostdb_entry = Ptr<HostDBInfo>(r);
2110     
   CID 1364117:    (FORWARD_NULL)
   Assigning: "client_addr" = "NULL".
2111       sockaddr const *client_addr = NULL;
2112       bool use_client_addr        = t_state.http_config_param->use_client_target_addr == 1 && t_state.client_info.is_transparent &&
2113                              t_state.dns_info.os_addr_style == HttpTransact::DNSLookupInfo::OS_ADDR_TRY_DEFAULT;
2114       if (use_client_addr) {
2115         NetVConnection *vc = t_state.state_machine->ua_session ? t_state.state_machine->ua_session->get_netvc() : NULL;
2116         if (vc) {
/proxy/http/HttpSM.cc: 2111 in HttpSM::process_hostdb_info(HostDBInfo *)()
2105     void
2106     HttpSM::process_hostdb_info(HostDBInfo *r)
2107     {
2108       // Increment the refcount to our item, since we are pointing at it
2109       t_state.hostdb_entry = Ptr<HostDBInfo>(r);
2110     
   CID 1364117:    (FORWARD_NULL)
   Assigning: "client_addr" = "NULL".
2111       sockaddr const *client_addr = NULL;
2112       bool use_client_addr        = t_state.http_config_param->use_client_target_addr == 1 && t_state.client_info.is_transparent &&
2113                              t_state.dns_info.os_addr_style == HttpTransact::DNSLookupInfo::OS_ADDR_TRY_DEFAULT;
2114       if (use_client_addr) {
2115         NetVConnection *vc = t_state.state_machine->ua_session ? t_state.state_machine->ua_session->get_netvc() : NULL;
2116         if (vc) {

Issue Links

• relates to: TS-4938 Crash due to null client_vc
• links to: GitHub Pull Request #1092