[LOGCXX-250] SocketImpl will throw NPE if closed() while starting up

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.10.0
• Fix Version/s: 0.10.0
• Assignee: Curt Arnold
• Reporter: Curt Arnold
• Created: 12/Mar/08 21:47
• Updated: 31/Mar/08 15:25
• Resolved: 31/Mar/08 15:25

Description

SocketImpl does not check whether this->socket is null before making APR calls and does not synchronize access and modification of that value. This will result in a crash if TelnetAppender or similar is closed in start up or at some other inopportune time. This can be demonstrated by removing the Thread::sleep in TelnetAppenderTestCase::testActivateClose so that the appender is closed while the connection listening thread is being started.