Git-Mirrors/monero
1
0
Fork 0
mirror of https://github.com/monero-project/monero.git synced 2024-12-26 18:49:26 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #6244

Browse Source 
352bd132 abstract_tcp_server2: guard against negative timeouts (moneromooo-monero)
This commit is contained in:
Alexander Blair 2020-03-12 00:56:18 -07:00
commit 857abc368c
No known key found for this signature in database
GPG Key ID: C64552D877C32479
1 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
contrib/epee/include/net/abstract_tcp_server2.inl
View File

@ -363,8 +363,8 @@ PRAGMA_WARNING_DISABLE_VS(4355)
} }
delay *= 0.5; delay *= 0.5;
if (delay > 0) { long int ms = (long int)(delay * 100);
long int ms = (long int)(delay * 100); if (ms > 0) {
reset_timer(boost::posix_time::milliseconds(ms + 1), true); reset_timer(boost::posix_time::milliseconds(ms + 1), true);
boost::this_thread::sleep_for(boost::chrono::milliseconds(ms)); boost::this_thread::sleep_for(boost::chrono::milliseconds(ms));
} }
@ -721,7 +721,9 @@ PRAGMA_WARNING_DISABLE_VS(4355)
boost::posix_time::milliseconds connection<t_protocol_handler>::get_timeout_from_bytes_read(size_t bytes) boost::posix_time::milliseconds connection<t_protocol_handler>::get_timeout_from_bytes_read(size_t bytes)
{ {
boost::posix_time::milliseconds ms = (boost::posix_time::milliseconds)(unsigned)(bytes * TIMEOUT_EXTRA_MS_PER_BYTE); boost::posix_time::milliseconds ms = (boost::posix_time::milliseconds)(unsigned)(bytes * TIMEOUT_EXTRA_MS_PER_BYTE);
ms += m_timer.expires_from_now(); const auto cur = m_timer.expires_from_now().total_milliseconds();
if (cur > 0)
ms += (boost::posix_time::milliseconds)cur;
if (ms > get_default_timeout()) if (ms > get_default_timeout())
ms = get_default_timeout(); ms = get_default_timeout();
return ms; return ms;
@ -747,7 +749,12 @@ PRAGMA_WARNING_DISABLE_VS(4355)
template<class t_protocol_handler> template<class t_protocol_handler>
void connection<t_protocol_handler>::reset_timer(boost::posix_time::milliseconds ms, bool add) void connection<t_protocol_handler>::reset_timer(boost::posix_time::milliseconds ms, bool add)
{ {
MTRACE("Setting " << ms << " expiry"); if (ms.total_milliseconds() < 0)
{
MWARNING("Ignoring negative timeout " << ms);
return;
}
MTRACE((add ? "Adding" : "Setting") << " " << ms << " expiry");
auto self = safe_shared_from_this(); auto self = safe_shared_from_this();
if(!self) if(!self)
{ {
@ -760,7 +767,11 @@ PRAGMA_WARNING_DISABLE_VS(4355)
return; return;
} }
if (add) if (add)
ms += m_timer.expires_from_now(); {
const auto cur = m_timer.expires_from_now().total_milliseconds();
if (cur > 0)
ms += (boost::posix_time::milliseconds)cur;
}
m_timer.expires_from_now(ms); m_timer.expires_from_now(ms);
m_timer.async_wait([=](const boost::system::error_code& ec) m_timer.async_wait([=](const boost::system::error_code& ec)
{ {