select first hop for inbound tunnel from connected peers

TunnelPool.cpp

@@ -70,7 +70,7 @@ namespace tunnel
 			std::unique_lock<std::mutex> l(m_OutboundTunnelsMutex);
 			m_OutboundTunnels.insert (createdTunnel);
 		}
-		CreatePairedInboundTunnel (createdTunnel);
+		//CreatePairedInboundTunnel (createdTunnel);
 	}
 
 	void TunnelPool::TunnelExpired (std::shared_ptr<OutboundTunnel> expiredTunnel)
@@ -291,40 +291,50 @@ namespace tunnel
 		return hop;	
 	}	
 
-	void TunnelPool::CreateInboundTunnel ()
+	bool TunnelPool::SelectPeers (std::vector<std::shared_ptr<const i2p::data::RouterInfo> >& hops)
 	{
-		auto outboundTunnel = GetNextOutboundTunnel ();
-		if (!outboundTunnel)
-			outboundTunnel = tunnels.GetNextOutboundTunnel ();
-		LogPrint ("Creating destination inbound tunnel...");
 		auto prevHop = i2p::context.GetSharedRouterInfo ();	
-		std::vector<std::shared_ptr<const i2p::data::RouterInfo> > hops;
 		int numHops = m_NumInboundHops;
-		if (outboundTunnel)
+		if (i2p::transport::transports.GetNumPeers () > 25)
 		{
-			// last hop
+			auto r = i2p::transport::transports.GetRandomPeer ();
-			auto hop = outboundTunnel->GetTunnelConfig ()->GetFirstHop ()->router;
+			if (r && !r->GetProfile ()->IsBad ())
-			if (hop->GetIdentHash () != i2p::context.GetIdentHash ()) // outbound shouldn't be zero-hop tunnel
 			{
-				prevHop = hop;
+				prevHop = r;
-				hops.push_back (prevHop);
+				hops.push_back (r);
 				numHops--;
 			}
 		}
+		
 		for (int i = 0; i < numHops; i++)
 		{
 			auto hop = SelectNextHop (prevHop);
 			if (!hop)
 			{
-				LogPrint (eLogError, "Can't select next hop for inbound tunnel");
+				LogPrint (eLogError, "Can't select next hop");
-				return;
+				return false;
 			}	
 			prevHop = hop;
 			hops.push_back (hop);
 		}		
-		std::reverse (hops.begin (), hops.end ());	
+		return true;
-		auto tunnel = tunnels.CreateTunnel<InboundTunnel> (std::make_shared<TunnelConfig> (hops), outboundTunnel);
+	}	
-		tunnel->SetTunnelPool (shared_from_this ());
+		
+	void TunnelPool::CreateInboundTunnel ()
+	{
+		auto outboundTunnel = GetNextOutboundTunnel ();
+		if (!outboundTunnel)
+			outboundTunnel = tunnels.GetNextOutboundTunnel ();
+		LogPrint ("Creating destination inbound tunnel...");
+		std::vector<std::shared_ptr<const i2p::data::RouterInfo> > hops;
+		if (SelectPeers (hops))
+		{
+			std::reverse (hops.begin (), hops.end ());	
+			auto tunnel = tunnels.CreateTunnel<InboundTunnel> (std::make_shared<TunnelConfig> (hops), outboundTunnel);
+			tunnel->SetTunnelPool (shared_from_this ());
+		}	
+		else
+			LogPrint (eLogError, "Can't create inbound tunnel. No peers available");
 	}
 
 	void TunnelPool::RecreateInboundTunnel (std::shared_ptr<InboundTunnel> tunnel)
@@ -345,34 +355,15 @@ namespace tunnel
 		if (inboundTunnel)
 		{	
 			LogPrint ("Creating destination outbound tunnel...");
-			int numHops = m_NumOutboundHops;
-			auto prevHop = i2p::context.GetSharedRouterInfo ();
 			std::vector<std::shared_ptr<const i2p::data::RouterInfo> > hops;
-			if (i2p::transport::transports.GetNumPeers () > 25)
+			if (SelectPeers (hops))
 			{	
-				auto r = i2p::transport::transports.GetRandomPeer ();
+				auto tunnel = tunnels.CreateTunnel<OutboundTunnel> (
-				if (r)
+					std::make_shared<TunnelConfig> (hops, inboundTunnel->GetTunnelConfig ()));
-				{
+				tunnel->SetTunnelPool (shared_from_this ());
-					prevHop = r;
-					hops.push_back (r);
-					numHops--;
-				}
 			}	
-			for (int i = 0; i < numHops; i++)
+			else
-			{
+				LogPrint (eLogError, "Can't create outbound tunnel. No peers available");
-				auto hop = SelectNextHop (prevHop);
-				if (!hop)
-				{
-					LogPrint (eLogError, "Can't select next hop for outbound tunnel");
-					return;
-				}	
-				prevHop = hop;
-				hops.push_back (hop);
-			}	
-				
-			auto tunnel = tunnels.CreateTunnel<OutboundTunnel> (
-				std::make_shared<TunnelConfig> (hops, inboundTunnel->GetTunnelConfig ()));
-			tunnel->SetTunnelPool (shared_from_this ());
 		}	
 		else
 			LogPrint (eLogError, "Can't create outbound tunnel. No inbound tunnels found");

TunnelPool.h

@@ -61,6 +61,7 @@ namespace tunnel
 			template<class TTunnels>
 			typename TTunnels::value_type GetNextTunnel (TTunnels& tunnels, typename TTunnels::value_type excluded) const;
 			std::shared_ptr<const i2p::data::RouterInfo> SelectNextHop (std::shared_ptr<const i2p::data::RouterInfo> prevHop) const;
+			bool SelectPeers (std::vector<std::shared_ptr<const i2p::data::RouterInfo> >& hops);
 			
 		private: