Boost Asio,异步 UDP 客户端 - 关机时崩溃
Boost Asio, async UDP client - crash on shutdown
我在我的应用程序中为 IPC 使用 UDP 客户端-服务器
它工作正常,但是当尝试关闭客户端时会发生一些竞争情况,这会导致应用程序崩溃或死锁。
UDP 客户端:
// IOServiceBase class contain boost::asio::io_service instance
// it is accessible by service() protected method
class AsyncUDPClient : public IOServiceBase
{
public:
/// @brief Create a network client
AsyncUDPClient(const std::string& host, const std::string port)
: _host(host)
, _port(port)
, _reply()
, _work(service())
, _sock(service(), ba_ip::udp::endpoint(ba_ip::udp::v4(), 0)) {
run();
}
/// @brief Start async packets processing
void run(){
std::thread t([&]{ service().run(); });
t.detach();
}
/// @brief Async request to server
void send(uint8_t* message, size_t length) {
std::vector<uint8_t> packet(message, message + length);
service().post(boost::bind(&AsyncUDPClient::do_send, this, packet));
}
/// @brief Cleanup io_service to dismiss already putted tasks
~AsyncUDPClient() {
close();
// trying to wait until service is stopped, but it does not help
while (!service().stopped()){
std::this_thread::sleep_for(std::chrono::milliseconds(100));
}
}
/// @brief Cleanup io_service to dismiss already putted tasks
void close(){
std::thread t([&]{ service().stop(); });
t.join();
}
protected:
// send-response methods are pretty standard
private:
std::string _host;
std::string _port;
std::array<uint8_t, max_length> _reply;
ba::io_service::work _work;
ba_ip::udp::socket _sock;
};
用法示例:
{
AsyncUDPClient ipc(addr, port);
ipc.send(&archive_data[0], archive_data.size());
// it seems client is destroyed before some internal processing is finished?
}
行为不是确定性的,有时工作正常,有时崩溃,有时死机。
Stacktrace 在 boost.asio 内部
的某处显示崩溃点
AsyncUDPClient
的破坏未与线程 运行 和 io_service
正确同步。当处理 io_service
的线程在其生命周期结束后尝试与 AsyncUDPClient
及其 io_service
交互时,这可能会导致调用未定义的行为。
要解决此问题,请不要从处理 io_service
的线程分离,并在 io_service
停止后显式加入线程。
class AsyncUDPClient
: public IOServiceBase
{
public:
// ...
void run()
{
_threads.emplace_back([&]{ service().run(); })
}
// ...
~AsyncUDPClient()
{
close();
}
void close()
{
// Stop the io_service. This changes its state and return immediately.
service().stop();
// Explicitly synchronize with threads running the io_service.
for (auto& thread: _threads)
{
thread.join();
}
}
private:
// ...
std::vector<std::thread> _threads;
};
正如上面评论中所暗示的,io_service::stop()
does not block. It changes the state of io_service
to stopped, returns immediately, and causes all invocations of run()
and run_one()
to return as soon as possible. Calls to io_service::stopped()
立即 return io_service
的状态。这些调用均未指示 run()
或 run_one()
.
的调用中当前是否存在线程
我在我的应用程序中为 IPC 使用 UDP 客户端-服务器
它工作正常,但是当尝试关闭客户端时会发生一些竞争情况,这会导致应用程序崩溃或死锁。
UDP 客户端:
// IOServiceBase class contain boost::asio::io_service instance
// it is accessible by service() protected method
class AsyncUDPClient : public IOServiceBase
{
public:
/// @brief Create a network client
AsyncUDPClient(const std::string& host, const std::string port)
: _host(host)
, _port(port)
, _reply()
, _work(service())
, _sock(service(), ba_ip::udp::endpoint(ba_ip::udp::v4(), 0)) {
run();
}
/// @brief Start async packets processing
void run(){
std::thread t([&]{ service().run(); });
t.detach();
}
/// @brief Async request to server
void send(uint8_t* message, size_t length) {
std::vector<uint8_t> packet(message, message + length);
service().post(boost::bind(&AsyncUDPClient::do_send, this, packet));
}
/// @brief Cleanup io_service to dismiss already putted tasks
~AsyncUDPClient() {
close();
// trying to wait until service is stopped, but it does not help
while (!service().stopped()){
std::this_thread::sleep_for(std::chrono::milliseconds(100));
}
}
/// @brief Cleanup io_service to dismiss already putted tasks
void close(){
std::thread t([&]{ service().stop(); });
t.join();
}
protected:
// send-response methods are pretty standard
private:
std::string _host;
std::string _port;
std::array<uint8_t, max_length> _reply;
ba::io_service::work _work;
ba_ip::udp::socket _sock;
};
用法示例:
{
AsyncUDPClient ipc(addr, port);
ipc.send(&archive_data[0], archive_data.size());
// it seems client is destroyed before some internal processing is finished?
}
行为不是确定性的,有时工作正常,有时崩溃,有时死机。 Stacktrace 在 boost.asio 内部
的某处显示崩溃点AsyncUDPClient
的破坏未与线程 运行 和 io_service
正确同步。当处理 io_service
的线程在其生命周期结束后尝试与 AsyncUDPClient
及其 io_service
交互时,这可能会导致调用未定义的行为。
要解决此问题,请不要从处理 io_service
的线程分离,并在 io_service
停止后显式加入线程。
class AsyncUDPClient
: public IOServiceBase
{
public:
// ...
void run()
{
_threads.emplace_back([&]{ service().run(); })
}
// ...
~AsyncUDPClient()
{
close();
}
void close()
{
// Stop the io_service. This changes its state and return immediately.
service().stop();
// Explicitly synchronize with threads running the io_service.
for (auto& thread: _threads)
{
thread.join();
}
}
private:
// ...
std::vector<std::thread> _threads;
};
正如上面评论中所暗示的,io_service::stop()
does not block. It changes the state of io_service
to stopped, returns immediately, and causes all invocations of run()
and run_one()
to return as soon as possible. Calls to io_service::stopped()
立即 return io_service
的状态。这些调用均未指示 run()
或 run_one()
.