#6526 Fixed crash when socket is null on handling TCP connection, and added some logging

This commit is contained in:
walker0643 2018-02-01 11:18:13 -05:00 committed by Jnewbon
parent 30c6b3a588
commit b3a90dddae
2 changed files with 10 additions and 0 deletions

View File

@ -869,5 +869,9 @@ SecureSocket::showSecureConnectInfo()
void
SecureSocket::handleTCPConnected(const Event& event, void*)
{
if (getSocket() == nullptr) {
LOG((CLOG_DEBUG "disregarding stale connect event"));
return;
}
secureConnect();
}

View File

@ -51,6 +51,8 @@ TCPSocket::TCPSocket(IEventQueue* events, SocketMultiplexer* socketMultiplexer,
throw XSocketCreate(e.what());
}
LOG((CLOG_DEBUG "Opening new socket: %08X", m_socket));
init();
}
@ -64,6 +66,8 @@ TCPSocket::TCPSocket(IEventQueue* events, SocketMultiplexer* socketMultiplexer,
{
assert(m_socket != NULL);
LOG((CLOG_DEBUG "Opening new socket: %08X", m_socket));
// socket starts in connected state
init();
onConnected();
@ -97,6 +101,8 @@ TCPSocket::bind(const NetworkAddress& addr)
void
TCPSocket::close()
{
LOG((CLOG_DEBUG "Closing socket: %08X", m_socket));
// remove ourself from the multiplexer
setJob(NULL);