This commit is contained in:
AlexBeLi 2016-10-03 16:05:51 +00:00 committed by GitHub
commit 75b7f10466
2 changed files with 10 additions and 12 deletions

View file

@ -171,7 +171,7 @@ namespace client
std::shared_ptr<const i2p::data::LeaseSet> LeaseSetDestination::FindLeaseSet (const i2p::data::IdentHash& ident) std::shared_ptr<const i2p::data::LeaseSet> LeaseSetDestination::FindLeaseSet (const i2p::data::IdentHash& ident)
{ {
std::lock_guard<std::mutex> lock(m_RemoteLeaseSetsMutex); std::unique_lock<std::mutex> lock(m_RemoteLeaseSetsMutex);
auto it = m_RemoteLeaseSets.find (ident); auto it = m_RemoteLeaseSets.find (ident);
if (it != m_RemoteLeaseSets.end ()) if (it != m_RemoteLeaseSets.end ())
{ {
@ -185,10 +185,7 @@ namespace client
if(ls && !ls->IsExpired()) if(ls && !ls->IsExpired())
{ {
ls->PopulateLeases(); ls->PopulateLeases();
{ m_RemoteLeaseSets[ident] = ls;
std::lock_guard<std::mutex> _lock(m_RemoteLeaseSetsMutex);
m_RemoteLeaseSets[ident] = ls;
}
} }
}); });
} }
@ -203,15 +200,12 @@ namespace client
if (ls && !ls->IsExpired ()) if (ls && !ls->IsExpired ())
{ {
ls->PopulateLeases (); // since we don't store them in netdb ls->PopulateLeases (); // since we don't store them in netdb
{ m_RemoteLeaseSets[ident] = ls;
std::lock_guard<std::mutex> lock(m_RemoteLeaseSetsMutex);
m_RemoteLeaseSets[ident] = ls;
}
return ls; return ls;
} }
} }
return nullptr; return nullptr;
} }
std::shared_ptr<const i2p::data::LocalLeaseSet> LeaseSetDestination::GetLeaseSet () std::shared_ptr<const i2p::data::LocalLeaseSet> LeaseSetDestination::GetLeaseSet ()
{ {

View file

@ -103,7 +103,8 @@ namespace client
} }
auto path = remoteSession->GetSharedRoutingPath (); auto path = remoteSession->GetSharedRoutingPath ();
std::shared_ptr<i2p::tunnel::OutboundTunnel> outboundTunnel; std::shared_ptr<i2p::tunnel::OutboundTunnel> outboundTunnel;
std::shared_ptr<const i2p::data::Lease> remoteLease; std::shared_ptr<const i2p::data::Lease> remoteLease;
bool unconfirmedTags=false;
if (path) if (path)
{ {
if (!remoteSession->CleanupUnconfirmedTags ()) // no stuck tags if (!remoteSession->CleanupUnconfirmedTags ()) // no stuck tags
@ -112,9 +113,12 @@ namespace client
remoteLease = path->remoteLease; remoteLease = path->remoteLease;
} }
else else
{
remoteSession->SetSharedRoutingPath (nullptr); remoteSession->SetSharedRoutingPath (nullptr);
unconfirmedTags=true;
}
} }
else if (!path || unconfirmedTags)
{ {
outboundTunnel = GetTunnelPool ()->GetNextOutboundTunnel (); outboundTunnel = GetTunnelPool ()->GetNextOutboundTunnel ();
auto leases = remote->GetNonExpiredLeases (); auto leases = remote->GetNonExpiredLeases ();