aboutsummaryrefslogtreecommitdiff
path: root/contrib/epee/src/network_throttle-detail.cpp
diff options
context:
space:
mode:
authormoneromooo-monero <moneromooo-monero@users.noreply.github.com>2017-12-13 18:46:03 +0000
committermoneromooo-monero <moneromooo-monero@users.noreply.github.com>2017-12-18 15:16:11 +0000
commitbd5cce07b32efade012a4a8872ea7dd6d3abf1ed (patch)
tree1477823fe0a6ba1f3617be69a44d6f0788d5b6af /contrib/epee/src/network_throttle-detail.cpp
parentnetwork_throttle: remove unused xxx static member (diff)
downloadmonero-bd5cce07b32efade012a4a8872ea7dd6d3abf1ed.tar.xz
network_throttle: fix ineffective locking
Diffstat (limited to 'contrib/epee/src/network_throttle-detail.cpp')
-rw-r--r--contrib/epee/src/network_throttle-detail.cpp8
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: