Author: tross
Date: Wed Sep  2 14:45:33 2009
New Revision: 810532

URL: http://svn.apache.org/viewvc?rev=810532&view=rev
Log:
Fixed a race condition whereby the connection thread sometimes read the value of
"delayMin" before it was initialized by the constructor.

Modified:
    qpid/trunk/qpid/cpp/src/qmf/ResilientConnection.cpp

Modified: qpid/trunk/qpid/cpp/src/qmf/ResilientConnection.cpp
URL: 
http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/src/qmf/ResilientConnection.cpp?rev=810532&r1=810531&r2=810532&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/src/qmf/ResilientConnection.cpp (original)
+++ qpid/trunk/qpid/cpp/src/qmf/ResilientConnection.cpp Wed Sep  2 14:45:33 2009
@@ -176,7 +176,7 @@
 }
 
 ResilientConnectionImpl::ResilientConnectionImpl(const ConnectionSettings& 
_settings) :
-    notifyFd(-1), connected(false), shutdown(false), settings(_settings), 
connThread(*this)
+    notifyFd(-1), connected(false), shutdown(false), settings(_settings), 
delayMin(1), connThread(*this)
 {
     
connection.registerFailureCallback(boost::bind(&ResilientConnectionImpl::failure,
 this));
     settings.impl->getRetrySettings(&delayMin, &delayMax, &delayFactor);



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscr...@qpid.apache.org

Reply via email to