slightly safer connection semaphore management

1 files changed, 3 insertions, 1 deletions

diff --git a/util/message_server_port.cpp b/util/message_server_port.cpp
index 698359749e4..6d00628c8ad 100644
--- a/util/message_server_port.cpp
+++ b/util/message_server_port.cpp
@@ -32,10 +32,11 @@ namespace mongo {
         MessageHandler * handler;
 
         void threadRun( MessagingPort * inPort) {
+            TicketHolderReleaser connTicketReleaser( &connTicketHolder );
+            
             assert( inPort );
 
             setThreadName( "conn" );
-            TicketHolderReleaser connTicketReleaser( &connTicketHolder );
 
             auto_ptr<MessagingPort> p( inPort );
 
@@ -101,6 +102,7 @@ namespace mongo {
                 boost::thread thr( boost::bind( &pms::threadRun , p ) );
             }
             catch ( boost::thread_resource_error& ) {
+                connTicketHolder.release();
                 log() << "can't create new thread, closing connection" << endl;
 
                 p->shutdown();