From 95a23dc7ec9a2ce55cab337b78b662a30ec2b986 Mon Sep 17 00:00:00 2001
From: Adam Ierymenko <adam.ierymenko@gmail.com>
Date: Fri, 9 Aug 2013 17:20:40 -0400
Subject: Fix for another wonderful C++ threading race condition.

---
 node/Network.cpp | 4 ++++
 1 file changed, 4 insertions(+)

(limited to 'node/Network.cpp')

diff --git a/node/Network.cpp b/node/Network.cpp
index 9227cd34..3593b732 100644
--- a/node/Network.cpp
+++ b/node/Network.cpp
@@ -139,6 +139,7 @@ SharedPtr<Network> Network::newInstance(const RuntimeEnvironment *renv,uint64_t
 	// that then causes the Network instance to be deleted before it is finished
 	// being constructed. C++ edge cases, how I love thee.
 	SharedPtr<Network> nw(new Network());
+	nw->_ready = false; // disable handling of Ethernet frames during construct
 	nw->_r = renv;
 	nw->_rlLimit.bytesPerSecond = ZT_MULTICAST_DEFAULT_BYTES_PER_SECOND;
 	nw->_rlLimit.maxBalance = ZT_MULTICAST_DEFAULT_RATE_MAX_BALANCE;
@@ -150,6 +151,7 @@ SharedPtr<Network> Network::newInstance(const RuntimeEnvironment *renv,uint64_t
 	if (nw->controller() == renv->identity.address()) // sanity check, this isn't supported for now
 		throw std::runtime_error("cannot add a network for which I am the netconf master");
 	nw->_restoreState();
+	nw->_ready = true; // enable handling of Ethernet frames
 	nw->requestConfiguration();
 	return nw;
 }
@@ -267,6 +269,8 @@ Network::Status Network::status() const
 
 void Network::_CBhandleTapData(void *arg,const MAC &from,const MAC &to,unsigned int etherType,const Buffer<4096> &data)
 {
+	if (!((Network *)arg)->_ready)
+		return;
 	const RuntimeEnvironment *_r = ((Network *)arg)->_r;
 	try {
 		_r->sw->onLocalEthernet(SharedPtr<Network>((Network *)arg),from,to,etherType,data);
-- 
cgit v1.2.3