[TS-4141] CID 1348540: Null pointer dereferences

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 6.1.0
• Component/s: HTTP
• Labels: coverity
• Assignee: Susan Hinrichs
• Reporter: Leif Hedstrom
• Created: 17/Jan/16 21:02
• Updated: 04/Feb/16 18:49
• Resolved: 18/Jan/16 04:49

Description

*** CID 1348540:  Null pointer dereferences  (FORWARD_NULL)
/proxy/http/HttpSM.cc: 5643 in HttpSM::attach_server_session(HttpServerSession *)()
5637       server_entry->vc_handler = &HttpSM::state_send_server_request_header;
5638     
5639     
5640       // es - is this a concern here in HttpSM?  Does it belong somewhere else?
5641       // Get server and client connections
5642       UnixNetVConnection *server_vc = dynamic_cast<UnixNetVConnection *>(server_session->get_netvc());
   CID 1348540:  Null pointer dereferences  (FORWARD_NULL)
   Passing null pointer "this->ua_session" to "get_netvc", which dereferences it. (The dereference happens because this is a virtual function call.)
5643       UnixNetVConnection *client_vc = (UnixNetVConnection *)(ua_session->get_netvc());
5644       SSLNetVConnection *ssl_vc = dynamic_cast<SSLNetVConnection *>(client_vc);
5645       bool associated_connection = false;
5646       if (server_vc) { // if server_vc isn't a PluginVC
5647         if (ssl_vc) {  // if incoming connection is SSL
5648           bool client_trace = ssl_vc->getSSLTrace();