diff options
| author | Carl Hetherington <cth@carlh.net> | 2022-02-24 19:47:19 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2022-02-25 08:00:51 +0100 |
| commit | 5ec4efca61906608e2e7a590b3d1c3f2b7078e81 (patch) | |
| tree | ec5b593686870a321daea0ae02037917fb41a59b /src/lib/encode_server_finder.h | |
| parent | 76e2d6390305eeb59e4934ca66013b09b4a44dda (diff) | |
Fix some dubious thread/locking behaviour.
Previously we had server_found(), which took the lock and found
a server, which it returned as an iterator into the list.
However, it then released the lock, which I think left the
iterator unprotected.
This wasn't done in response to any particular bug, I just
noticed it on the way past.
Diffstat (limited to 'src/lib/encode_server_finder.h')
| -rw-r--r-- | src/lib/encode_server_finder.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/lib/encode_server_finder.h b/src/lib/encode_server_finder.h index 533c9219b..efb498880 100644 --- a/src/lib/encode_server_finder.h +++ b/src/lib/encode_server_finder.h @@ -66,7 +66,6 @@ private: void search_thread (); void listen_thread (); - boost::optional<std::list<EncodeServerDescription>::iterator> server_found (std::string); void start_accept (); void handle_accept (boost::system::error_code ec, std::shared_ptr<Socket> socket); |