Skip to content

Commit 54b980f

Browse files
randallzwoop
authored andcommitted
Adds null check (#6994)
This attempts to fix null pointer access introduced in 3ffd8ac (cherry picked from commit cd7d9ff)
1 parent 527de3b commit 54b980f

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

proxy/http/HttpSM.cc

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1809,10 +1809,13 @@ HttpSM::state_http_server_open(int event, void *data)
18091809
case VC_EVENT_ACTIVE_TIMEOUT:
18101810
case VC_EVENT_ERROR:
18111811
case NET_EVENT_OPEN_FAILED: {
1812-
NetVConnection *vc = server_session->get_netvc();
1813-
if (vc) {
1814-
server_connection_provided_cert = vc->provided_cert();
1812+
if (server_session) {
1813+
NetVConnection *vc = server_session->get_netvc();
1814+
if (vc) {
1815+
server_connection_provided_cert = vc->provided_cert();
1816+
}
18151817
}
1818+
18161819
t_state.current.state = HttpTransact::CONNECTION_ERROR;
18171820
// save the errno from the connect fail for future use (passed as negative value, flip back)
18181821
t_state.current.server->set_connect_fail(event == NET_EVENT_OPEN_FAILED ? -reinterpret_cast<intptr_t>(data) : ECONNABORTED);

0 commit comments

Comments
 (0)