diff options
| author | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2017-12-13 18:46:03 +0000 |
|---|---|---|
| committer | moneromooo-monero <moneromooo-monero@users.noreply.github.com> | 2017-12-18 15:16:11 +0000 |
| commit | bd5cce07b32efade012a4a8872ea7dd6d3abf1ed (patch) | |
| tree | 1477823fe0a6ba1f3617be69a44d6f0788d5b6af | |
| parent | network_throttle: remove unused xxx static member (diff) | |
| download | monero-bd5cce07b32efade012a4a8872ea7dd6d3abf1ed.tar.xz | |
network_throttle: fix ineffective locking
| -rw-r--r-- | contrib/epee/src/network_throttle-detail.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/contrib/epee/src/network_throttle-detail.cpp b/contrib/epee/src/network_throttle-detail.cpp index 317dde8e0..ed6bc07ed 100644 --- a/contrib/epee/src/network_throttle-detail.cpp +++ b/contrib/epee/src/network_throttle-detail.cpp @@ -231,8 +231,10 @@ network_time_seconds network_throttle::get_sleep_time_after_tick(size_t packet_s } void network_throttle::logger_handle_net(const std::string &filename, double time, size_t size) { - boost::mutex mutex; - mutex.lock(); { + static boost::mutex mutex; + + boost::lock_guard<boost::mutex> lock(mutex); + { std::fstream file; file.open(filename.c_str(), std::ios::app | std::ios::out ); file.precision(6); @@ -240,7 +242,7 @@ void network_throttle::logger_handle_net(const std::string &filename, double tim _warn("Can't open file " << filename); file << static_cast<int>(time) << " " << static_cast<double>(size/1024) << "\n"; file.close(); - } mutex.unlock(); + } } // fine tune this to decide about sending speed: |